OpenROAD icon indicating copy to clipboard operation
OpenROAD copied to clipboard

grt: update user defined reductions

Open eder-matheus opened this issue 3 years ago • 5 comments

eder-matheus avatar Aug 18 '22 22:08 eder-matheus

@eder-matheus what is the status here?

maliberty avatar Sep 06 '22 17:09 maliberty

@eder-matheus what is the status here?

This is ready to merge. I think we forgot it between the other tasks. I'll just solve the conflicts.

eder-matheus avatar Sep 13 '22 23:09 eder-matheus

There are test failures that need fixing too

maliberty avatar Sep 14 '22 20:09 maliberty

There are test failures that need fixing too

Now it's ready :)

eder-matheus avatar Sep 15 '22 14:09 eder-matheus

Has this been through secure CI?

maliberty avatar Sep 19 '22 04:09 maliberty

@maliberty This branch has two failures in the public-ci: ibex asap7: [ERROR] cts__design__instance__count__hold_buffer fail test: 15.0 <= 10.0 microwatt sky130hd: [ERROR] finish__timing__drv__max_slew_limit fail test: -2.0299 >= -1.58

However, the secure-ci is green, with a great improvement in ca53. The detailed route ends with 26 iterations, while in the master branch it ends with 50 iterations.

I don't understand why we have more cts buffers yet. I imagine that the global placement changed a lot due to the new groute resources. And if I remember correctly, the slew metric has a lot of noise. Should I investigate deeper these two failures, or can we just update these metrics?

eder-matheus avatar Oct 06 '22 22:10 eder-matheus

Updating is fine. ca53 is great!

maliberty avatar Oct 06 '22 22:10 maliberty

PR for the metrics update here: https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/pull/606

eder-matheus avatar Oct 07 '22 20:10 eder-matheus