-
Notifications
You must be signed in to change notification settings - Fork 414
Add a new API partition_edges() to RRGraphBuilder #1900
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a new API partition_edges() to RRGraphBuilder #1900
Conversation
@hariszafar-lm Please merge the master into this branch. This should clear the red flag. |
QoR tests are completed and a quick summary is given below. QoR comparison files are also attached. titan_quick_qor test: vtr_reg_qor_chain_depop test: Here are the attached files |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Once CI is green, we can merge.
@vaughnbetz If you see anything wrong, just let me know.
@hariszafar-lm Just need to resolve the conflicts. We are good to go. |
VtR Nightly test 1 is passing locally. Don't know why it's failing here. |
No worries. I will restart it. It is mainly due to some glitches in servers. |
@hariszafar-lm Just resolve the merging conflicts. We can merge now. |
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 refactored/deprecate the direct use of the legacy API
partition_edges()
from therr_node
data structure.After this PR, the
partition_edges()
from the refactored data structureRRGraphBuilder
is the only way to use it.Checklist:
partition_edges()
fromrr_node.cpp
andrr_node.h
(if present)partition_edges()
to data structuresRRGraphBuilder
, whose comments are Doxygen compatiblepartition_edges()
in respective functions (if needed)Related Issue
This pull request is a follow-up PR on the routing resource graph refactoring effort #1805, #1868
Motivation and Context
This PR is a continuation of the refactoring effort with a focus on shadowing the
rr_graph_storage
APIs in theRRGraphBuilder
data structure.This PR refactored the
partition_edges()
API among the other APIs in #1847, #1868How Has This Been Tested?
Types of changes
Checklist: