Skip to content

Conversation

@openroad-ci
Copy link
Collaborator

@openroad-ci openroad-ci commented Feb 11, 2026

Missing metric updates for #3879

designs/gf12/jpeg/rules-base.json updates:

Metric Old New Type
placeopt__design__instance__count__stdcell 86979 102448 Failing
cts__timing__setup__ws -38.5 -101.0 Failing
cts__timing__setup__tns -154.0 -22300.0 Failing
cts__timing__hold__ws -38.5 -25.0 Tighten
cts__timing__hold__tns -154.0 -100.0 Tighten
globalroute__timing__setup__ws -38.5 -118.0 Failing
globalroute__timing__setup__tns -154.0 -11500.0 Failing
globalroute__timing__hold__ws -38.5 -25.0 Tighten
globalroute__timing__hold__tns -154.0 -100.0 Tighten
finish__timing__setup__ws -38.5 -103.0 Failing
finish__timing__setup__tns -154.0 -812.0 Failing
finish__timing__hold__ws -38.5 -25.0 Tighten
finish__timing__hold__tns -154.0 -100.0 Tighten

CI Could not Update Rules

[ERROR] asap7/mock-cpu: No metrics found, please check the latest CI build for issues.

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

@maliberty maliberty left a comment

Choose a reason for hiding this comment

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

I'm surprised that a 5% increase in core util had a noticeably negative effect on final timing.

@maliberty maliberty enabled auto-merge February 11, 2026 18:03
@eder-matheus
Copy link
Member

I'm surprised that a 5% increase in core util had a noticeably negative effect on final timing.

I also updated clock period from 770 to 500. The core increase alone actually improved timing, so I changed the clock period to have negative slack.

@eder-matheus eder-matheus added the UpdateRules Starts GHA to update rules label Feb 11, 2026
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label Feb 11, 2026
@maliberty maliberty merged commit e05a2d1 into The-OpenROAD-Project:master Feb 12, 2026
8 checks passed
@maliberty maliberty deleted the fix_metrics branch February 12, 2026 00:01
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