Skip to content

update metrics gcd asap7 - #4481

Open
jfgava wants to merge 2 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:update-metrics-full-res-aware-grt
Open

update metrics gcd asap7#4481
jfgava wants to merge 2 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:update-metrics-full-res-aware-grt

Conversation

@jfgava

@jfgava jfgava commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

designs/asap7/gcd/rules-base.json updates:

Metric Old New Type
placeopt__design__instance__count__stdcell 476 473 Tighten
cts__timing__setup__ws -58.6 -57.3 Tighten
globalroute__timing__setup__tns -206.0 -282.0 Failing
detailedroute__route__wirelength 919 908 Tighten
finish__timing__setup__ws -54.4 -53.7 Tighten

jfgava added 2 commits August 27, 2026 21:53
Signed-off-by: Jonas Gava <jfgava@precisioninno.com>
Signed-off-by: Jonas Gava <jfgava@precisioninno.com>
@jfgava jfgava self-assigned this Aug 27, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the OpenROAD subproject commit and adjusts several baseline design rules and timing metrics in rules-base.json. Feedback highlights a significant degradation in the global routing setup TNS (globalroute__timing__setup__tns), suggesting a need to analyze the global routing stage for potential congestion or optimization issues rather than just updating the threshold.

},
"globalroute__timing__setup__tns": {
"value": -206.0,
"value": -282.0,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The timing metric 'globalroute__timing__setup__tns' shows a significant degradation of approximately 37% (from -206.0 to -282.0). While other timing metrics like 'cts__timing__setup__ws' and 'finish__timing__setup__ws' have improved, this large degradation in global routing TNS suggests potential routing congestion or optimization issues introduced by the OpenROAD update. It is recommended to analyze the global routing stage to identify the cause of this regression rather than simply relaxing the regression threshold.

@jfgava

jfgava commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

Related to PR #11255

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant