grt: add CUGR global_route_debug -st visualization support#10121
grt: add CUGR global_route_debug -st visualization support#10121hansb21 wants to merge 1 commit intoThe-OpenROAD-Project:masterfrom
Conversation
There was a problem hiding this comment.
Welcome to OpenROAD! Thanks for opening your first PR.
Before we review:
- Contribution Guide: https://openroad.readthedocs.io/en/latest/contrib/contributing.html
- Build Instructions: https://openroad.readthedocs.io/en/latest/contrib/BuildWithCMake.html
Please ensure:
- CI passes
- Code is properly formatted
- Tests are included where applicable
A maintainer will review shortly!
There was a problem hiding this comment.
Code Review
This pull request introduces comprehensive debugging and visualization support for the CUGR global router, bringing it to parity with the FastRoute debug interface. Changes include new debug initialization and retrieval methods in the GlobalRouter class, updates to the SWIG interface files, and the implementation of Steiner tree visualization routines within the CUGR engine. The review feedback identifies several critical improvements: addressing potential dangling pointers by clearing the renderer after use, replacing hardcoded magic numbers with design-specific values for net properties, and refactoring duplicated FrNet initialization logic into a shared helper function.
7d1a18b to
53a0528
Compare
Signed-off-by: Hans Buss <hans.buss@mailfence.com>
3e9f35f to
5186387
Compare
Summary
This PR adds
global_route_debug -stvisualization support forCUGR, aligned with the existing FastRoute debug flow in GUI.
Main updates:
global_route_debug -stwhen running withglobal_route -use_cugr.(pattern routing).
Type of Change
Impact
Users can now visualize the Stage-1 Steiner tree for CUGR in GUI
using:
global_route_debug -net <net_name> -stglobal_route -use_cugrFastRoute debug behavior remains available.
Verification
(
./etc/Build.sh).Visual Evidence
Visual Evidence
FastRoute (
global_route_debug -net "net1707" -st+global_route)CUGR (
global_route_debug -net "net1707" -st+global_route -use_cugr)Related Issues
N/A