Add a new API set_node_direction() to RRGraphBuilder #1854
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR focuses on updating routing resource graph builder functions, where we use the refactored data structure
RRGraphBuilder
to shadow the discrete data structurerr_graph_storage
.This PR aims to fully deprecate the direct use of the legacy API
set_direction()
from therr_node
data structure.After this PR, the
set_direction
API is fully deprecated and theset_node_direction
from the refactored data structureRRGraphBuilder
is the only way to use it.Checklist:
set_direction()
fromrr_node.cpp
andrr_node.h
set_node_direction
to data structuresRRGraphBuilder
, whose comments are Doxygen compatibleset_direction()
in builder functions byset_node_direction()
Related Issue
This pull request is a follow-up PR on the routing resource graph refactoring effort #1805
Motivation and Context
This PR is the continuation of the refactoring effort with a focus on shadowing the
rr_graph_storage
APIs in theRRGraphBuilder
data structure.This PR reworks the
set_node_direction()
API among the other APIs in #1847How Has This Been Tested?
Types of changes
Checklist: