Skip to content

Commit

Permalink
Merge pull request verilog-to-routing#2745 from treelin611/rr_graphvi…
Browse files Browse the repository at this point in the history
…ew_doxygen

Add some overview to the RRGraphView doxygen documentation
  • Loading branch information
vaughnbetz authored Sep 27, 2024
2 parents 40797ae + 0b40272 commit e54189d
Show file tree
Hide file tree
Showing 3 changed files with 264 additions and 174 deletions.
6 changes: 3 additions & 3 deletions doc/src/api/vpr/rr_graph.rst
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ RRGraphView

.. doxygenfile:: rr_graph_view.h
:project: librrgraph
:sections: detaileddescription
:sections: briefdescription detaileddescription func prototype user-defined public-func

.. doxygenclass:: RRGraphView
:project: librrgraph
Expand All @@ -18,7 +18,7 @@ RRGraphBuilder

.. doxygenfile:: rr_graph_builder.h
:project: librrgraph
:sections: detaileddescription
:sections: briefdescription detaileddescription func prototype user-defined public-func

.. doxygenclass:: RRGraphBuilder
:project: librrgraph
Expand All @@ -29,7 +29,7 @@ RRSpatialLookup

.. doxygenfile:: rr_spatial_lookup.h
:project: librrgraph
:sections: detaileddescription
:sections: briefdescription detaileddescription func prototype user-defined public-func

.. doxygenclass:: RRSpatialLookup
:project: librrgraph
Expand Down
20 changes: 14 additions & 6 deletions libs/librrgraph/src/base/rr_graph_builder.h
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ class RRGraphBuilder {
t_rr_graph_storage& rr_nodes();
/** @brief Return a writable object for update the fast look-up of rr_node */
RRSpatialLookup& node_lookup();
/** .. warning:: The Metadata should stay as an independent data structure than rest of the internal data,
/** @warning The Metadata should stay as an independent data structure from the rest of the internal data,
* e.g., node_lookup! */
/** @brief Return a writable object for the meta data on the nodes */
MetadataStorage<int>& rr_node_metadata();
Expand Down Expand Up @@ -84,8 +84,12 @@ class RRGraphBuilder {

return segment_id;
}
/** TODO @brief Return a writable list of all the rr_segments
* .. warning:: It is not recommended to use this API unless you have to. The API may be deprecated later, and future APIs will designed to return a specific data from the rr_segments.
/**
* \internal
* TODO
* \endinternal
* @brief Return a writable list of all the rr_segments
* @warning It is not recommended to use this API unless you have to. The API may be deprecated later, and future APIs will designed to return a specific data from the rr_segments.
*/
inline vtr::vector<RRSegmentId, t_segment_inf>& rr_segments() {
return rr_segments_;
Expand All @@ -103,8 +107,12 @@ class RRGraphBuilder {

return switch_id;
}
/** TODO @brief Return a writable list of all the rr_switches
* .. warning:: It is not recommended to use this API unless you have to. The API may be deprecated later, and future APIs will designed to return a specific data from the rr_switches.
/**
* \internal
* TODO
* \endinternal
* @brief Return a writable list of all the rr_switches
* @warning It is not recommended to use this API unless you have to. The API may be deprecated later, and future APIs will designed to return a specific data from the rr_switches.
*/
inline vtr::vector<RRSwitchId, t_rr_switch_inf>& rr_switch() {
return rr_switch_inf_;
Expand Down Expand Up @@ -388,7 +396,7 @@ class RRGraphBuilder {
/* Detailed information about the switches, which are used in the RRGraph */
vtr::vector<RRSwitchId, t_rr_switch_inf> rr_switch_inf_;

/** .. warning:: The Metadata should stay as an independent data structure than rest of the internal data,
/** @warning The Metadata should stay as an independent data structure from the rest of the internal data,
* e.g., node_lookup! */
/* Metadata is an extra data on rr-nodes and edges, respectively, that is not used by vpr
* but simply passed through the flow so that it can be used by downstream tools.
Expand Down
Loading

0 comments on commit e54189d

Please sign in to comment.