Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update metrics due to FastRoute refactor #2476

Conversation

joaomai
Copy link

@joaomai joaomai commented Oct 17, 2024

Changes introduced to OpenRoad with the PR #5906 causes some designs to fail. The changes occur due to a change from a variable in mazeRouteMSMD from float to double.

aes_sky130hd

Metric Old New Type
globalroute__antenna_diodes_count 179 218 Failing
detailedroute__route__wirelength 678336 677310 Tighten
detailedroute__antenna_diodes_count 9 15 Failing

bp_be_nangate45

Metric Old New Type
finish__timing__setup__ws -0.59 -0.99 Failing
finish__timing__drv__hold_violation_count 100 775 Failing
finish__timing__wns_percent_delay -22.6 -46.18 Failing

chameleon_sky130hd

Metric Old New Type
detailedroute__antenna_diodes_count 45 54 Failing

microwatt_sky130hd
[WARNING] Multiple clocks not supported. Will use first clock: ext_clk: 15.0000.

Metric Old New Type
globalroute__antenna_diodes_count 7614 7292 Tighten
detailedroute__antenna__violating__nets 0 1 Failing
detailedroute__antenna_diodes_count 891 1048 Failing

risv32i_sky130hd

Metric Old New Type
placeopt__design__instance__area 83828 83236 Tighten
detailedroute__route__wirelength 337654 333349 Tighten
detailedroute__antenna_diodes_count 8 13 Failing
finish__timing__setup__ws -1.42 -1.31 Tighten
finish__design__instance__area 90442 90213 Tighten
finish__timing__wns_percent_delay -32.76 -30.94 Tighten

@joaomai joaomai requested a review from eder-matheus October 17, 2024 15:18
…w-scripts into update-metrics-fastroute-refactor
@eder-matheus eder-matheus merged commit e30d4e9 into The-OpenROAD-Project:master Oct 17, 2024
6 checks passed
@joaomai joaomai deleted the update-metrics-fastroute-refactor branch October 18, 2024 00: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.

2 participants