search

The-OpenROAD-Project / OpenROAD-flow-scripts

OpenROAD's scripts implementing an RTL-to-GDS Flow. Documentation at https://openroad-flow-scripts.readthedocs.io/en/latest/
https://theopenroadproject.org/
Other
342 stars 290 forks source link

update metrics for public designs #2383

Closed eder-matheus closed 1 month ago

eder-matheus commented 1 month ago
nangate45/ariane136 Metric Old New Type
detailedrouteroutewirelength 7826381 9268350 Failing
finishtimingdrv__hold_violation_count 858 101 Tighten
asap7/swerv_wrapper Metric Old New Type
detailedrouteroutedrc_errors 0 2 Failing
finishtimingsetup__ws -525.1 0.0 Tighten
finishtimingdrv__hold_violation_count 768 1111 Failing
finishtimingwns_percent_delay -23.31 -10.0 Tighten

Issue https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/issues/2382 created to track the 2 DRCs in asap7/swerv_wrapper.

osamahammad21 commented 1 month ago

Is it ok to merge with the 2 DRCs or should we handle them first? Also the wirelength increase is pretty high.

eder-matheus commented 1 month ago

Is it ok to merge with the 2 DRCs or should we handle them first? Also the wirelength increase is pretty high.

The wire length is due to macro placement changes, similar to what happened to nangate45/swerv_wrapper. The two DRCs are tracked in the issue mentioned in the PR description. I'm already looking at it, but since I don't have an ETA for the fix, I decided to create the PR to not have the CI broken.