Skip to content

update metrics after back-to-back jumpers fix#4314

Closed
jfgava wants to merge 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:update_metrics-sky130hd-fix-diodes
Closed

update metrics after back-to-back jumpers fix#4314
jfgava wants to merge 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:update_metrics-sky130hd-fix-diodes

Conversation

@jfgava

@jfgava jfgava commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Just some diodes metrics failing after last updates

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

Metric Old New Type
globalroute__antenna_diodes_count 1271 1276 Failing
detailedroute__antenna_diodes_count 1411 1341 Tighten
finish__timing__setup__ws -2.73 -2.69 Tighten
finish__timing__hold__ws -0.991 -0.989 Tighten
finish__design__instance__area 5575821 5575637 Tighten

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

Metric Old New Type
globalroute__antenna_diodes_count 207 209 Failing
detailedroute__route__wirelength 846595 846125 Tighten
finish__timing__setup__ws -0.282 -0.28 Tighten

Signed-off-by: Jonas Gava <jfgava@precisioninno.com>
@jfgava jfgava self-assigned this Jun 25, 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 submodule to a newer commit and adjusts the baseline design metrics (including antenna diode counts, wirelength, timing slack, and instance area) for the chameleon and microwatt designs on the sky130hd platform. No review comments were provided, so there is no feedback to address.

@maliberty

Copy link
Copy Markdown
Member

I've already got these in #4312

@maliberty maliberty closed this Jun 25, 2026
@jfgava

jfgava commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

I've already got these in #4312

Ok. Thanks

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.

2 participants