OpenROAD icon indicating copy to clipboard operation
OpenROAD copied to clipboard

gpl: modify routability driven default target RC value

Open openroad-robot opened this issue 1 year ago • 3 comments

With the previous default of 1.25 no design was being activated to run with routability driven option. With 1.0 we have routability driven mode working, resulting in less DRT iterations on a handful of designs.

openroad-robot avatar Feb 13 '24 00:02 openroad-robot

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Feb 13 '24 00:02 github-actions[bot]

@eder-matheus there is a secure CI running for this and so far it looks good. Please take a look tomorrow and see if it mergeable with some minor metrics updates.

maliberty avatar Feb 16 '24 03:02 maliberty

@eder-matheus there is a secure CI running for this and so far it looks good. Please take a look tomorrow and see if it mergeable with some minor metrics updates.

It has two metric failures, but ca53 has a timeout during DRT at the 46th iteration. In the master branch, ca53 ends with 30 iterations.

eder-matheus avatar Feb 16 '24 13:02 eder-matheus

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Mar 14 '24 02:03 github-actions[bot]

Secure-CI is finishing without errors.

gudeh avatar Apr 04 '24 13:04 gudeh

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Apr 04 '24 14:04 github-actions[bot]

Also, there was some metrics update on the secure-CI:

  • aes_lvt asap7 [ERROR] finish__timing__setup__ws fail test: -135.525 >= -105.98 [ERROR] finish__timing__wns_percent_delay fail test: -22.766924 >= -22.04

  • riscv32i asap7 [ERROR] finish__timing__setup__ws fail test: -553.269 >= -430.53 [ERROR] finish__timing__drv__setup_violation_count fail test: 1069.0 <= 1059.0

A PR for ORFS will be created after this PR is merged.

gudeh avatar Apr 04 '24 14:04 gudeh

A PR for ORFS will be created after this PR is merged.

Its good to have both ready so they can be merged at the same time. You only need to update the metrics in that PR.

maliberty avatar Apr 04 '24 15:04 maliberty