Skip to content

metric updates for gpl_runtime update#4205

Merged
maliberty merged 4 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-gpl_runtime
May 4, 2026
Merged

metric updates for gpl_runtime update#4205
maliberty merged 4 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-gpl_runtime

Conversation

@openroad-ci
Copy link
Copy Markdown
Collaborator

@openroad-ci openroad-ci commented May 2, 2026

designs/nangate45/mempool_group/rules-base.json updates:

Metric Old New Type
placeopt__design__instance__count__stdcell 188393 188355 Tighten
cts__timing__setup__tns -13300.0 -12000.0 Tighten
detailedroute__route__wirelength 5273763 5272257 Tighten
finish__timing__setup__tns -11500.0 -13800.0 Failing
finish__timing__hold__tns -0.738 -0.6 Tighten

designs/nangate45/bp_fe_top/rules-base.json updates:

Metric Old New Type
cts__timing__setup__tns -1.02 -0.752 Tighten
globalroute__timing__setup__tns -0.362 -0.36 Tighten
finish__timing__setup__ws -0.161 -0.15 Tighten
finish__timing__setup__tns -0.685 -1.58 Failing

designs/sky130hd/microwatt/rules-base.json updates:

Metric Old New Type
cts__timing__setup__tns -371.0 -345.0 Tighten
globalroute__antenna_diodes_count 1431 1426 Tighten
globalroute__timing__setup__tns -288.0 -322.0 Failing
detailedroute__antenna__violating__nets 6 5 Tighten
detailedroute__antenna_diodes_count 1688 1451 Tighten
finish__timing__setup__tns -145.0 -158.0 Failing

Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Copy link
Copy Markdown

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

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 tools/OpenROAD subproject commit from c5dc64a761155fd7323baf6990c639c9de20da19 to 146fe528b59420557f519feebdfcad5a70cd8384. I have no feedback to provide.

@maliberty
Copy link
Copy Markdown
Member

There are no metric updates here?

@eder-matheus eder-matheus added the UpdateRules Starts GHA to update rules label May 2, 2026
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label May 2, 2026
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
@eder-matheus eder-matheus added the UpdateRules Starts GHA to update rules label May 4, 2026
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label May 4, 2026
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
@eder-matheus eder-matheus requested a review from maliberty May 4, 2026 19:01
@eder-matheus
Copy link
Copy Markdown
Member

This should be ready to merge. UpdateRules didn't work, so I updated the metrics manually.

@maliberty maliberty enabled auto-merge May 4, 2026 19:59
@maliberty maliberty merged commit e91f812 into The-OpenROAD-Project:master May 4, 2026
7 of 8 checks passed
@maliberty maliberty deleted the secure-gpl_runtime branch May 4, 2026 21:19
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.

3 participants