OpenROAD-flow-scripts icon indicating copy to clipboard operation
OpenROAD-flow-scripts copied to clipboard
verified publisher icon The-OpenROAD-Project

Update OpenROAD submodule

Open github-actions[bot] opened this issue 10 months ago • 3 comments

Automated action to update tools/OpenROAD submodule and tighten CI rule checking. [ci:rules-tighten]

Updated Rules

designs/asap7/aes

Metric Old New Type
finish__timing__setup__ws -70.11 -62.23 Tighten

designs/asap7/aes-block

Metric Old New Type
finish__timing__setup__ws -119.26 -83.96 Tighten
finish__timing__wns_percent_delay -21.42 -20.89 Tighten

designs/asap7/aes-mbff

Metric Old New Type
finish__timing__setup__ws -68.71 -52.54 Tighten
finish__timing__wns_percent_delay -21.4 -17.87 Tighten

designs/asap7/aes_lvt

Metric Old New Type
finish__timing__setup__ws -29.62 -28.07 Tighten
finish__timing__wns_percent_delay -12.43 -12.06 Tighten

[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000. designs/asap7/ethmac

Metric Old New Type
finish__timing__setup__ws -187.76 -170.51 Tighten

[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000. designs/asap7/ethmac_lvt

Metric Old New Type
finish__timing__setup__ws -90.17 -86.2 Tighten

designs/asap7/gcd

Metric Old New Type
finish__design__instance__area 60 59 Tighten

designs/asap7/gcd-ccs

Metric Old New Type
finish__timing__setup__ws -96.02 -90.3 Tighten
finish__design__instance__area 57 56 Tighten
finish__timing__drv__setup_violation_count 43 42 Tighten

designs/asap7/ibex

Metric Old New Type
finish__timing__setup__ws -108.44 -89.94 Tighten
finish__timing__wns_percent_delay -15.17 -12.28 Tighten

designs/asap7/mock-alu

Metric Old New Type
finish__timing__setup__ws -515.34 -508.4 Tighten

designs/asap7/riscv32i

Metric Old New Type
detailedroute__route__wirelength 137800 137742 Tighten

designs/gf180/aes-hybrid

Metric Old New Type
finish__timing__setup__ws -1.3 -1.27 Tighten
finish__timing__wns_percent_delay -39.5 -38.65 Tighten

designs/gf180/ibex

Metric Old New Type
finish__timing__setup__ws -1.58 -1.35 Tighten
finish__timing__wns_percent_delay -21.3 -18.67 Tighten

designs/ihp-sg13g2/gcd

Metric Old New Type
finish__design__instance__area 27357 27346 Tighten

designs/ihp-sg13g2/ibex

Metric Old New Type
globalroute__antenna_diodes_count 1215 1074 Tighten
finish__timing__setup__ws -0.55 -0.5 Tighten

designs/ihp-sg13g2/jpeg

Metric Old New Type
detailedroute__antenna__violating__nets 110 99 Tighten
finish__timing__setup__ws -2.18 -1.92 Tighten
finish__timing__wns_percent_delay -30.54 -27.97 Tighten

designs/ihp-sg13g2/riscv32i

Metric Old New Type
detailedroute__antenna__violating__nets 30 17 Tighten

designs/ihp-sg13g2/spi

Metric Old New Type
finish__timing__setup__ws -0.3 -0.23 Tighten
finish__timing__drv__setup_violation_count 20 19 Tighten

designs/nangate45/aes

Metric Old New Type
finish__timing__setup__ws -0.14 -0.11 Tighten
finish__timing__wns_percent_delay -21.31 -18.03 Tighten

designs/nangate45/ariane133

Metric Old New Type
finish__timing__wns_percent_delay -11.66 -11.64 Tighten

designs/nangate45/bp_be_top

Metric Old New Type
finish__timing__setup__ws -0.76 -0.66 Tighten

designs/nangate45/bp_fe_top

Metric Old New Type
finish__timing__setup__ws -0.13 -0.11 Tighten
finish__timing__drv__hold_violation_count 1878 681 Tighten
finish__timing__wns_percent_delay -12.16 -11.45 Tighten

designs/nangate45/bp_multi_top

Metric Old New Type
finish__timing__drv__hold_violation_count 1026 440 Tighten

designs/nangate45/dynamic_node

Metric Old New Type
finish__timing__setup__ws -0.51 -0.48 Tighten
finish__timing__wns_percent_delay -33.08 -30.35 Tighten

designs/nangate45/gcd

Metric Old New Type
finish__timing__wns_percent_delay -25.2 -25.05 Tighten

designs/nangate45/ibex

Metric Old New Type
finish__timing__setup__ws -0.26 -0.2 Tighten
finish__timing__wns_percent_delay -18.58 -15.99 Tighten

designs/nangate45/swerv

Metric Old New Type
finish__timing__setup__ws -0.37 -0.3 Tighten

designs/nangate45/swerv_wrapper

Metric Old New Type
placeopt__design__instance__area 756670 756613 Tighten
finish__timing__setup__ws -0.53 -0.48 Tighten
finish__timing__drv__hold_violation_count 245 146 Tighten
finish__timing__wns_percent_delay -27.05 -25.32 Tighten

designs/nangate45/tinyRocket

Metric Old New Type
finish__timing__setup__ws -0.31 -0.26 Tighten
finish__timing__wns_percent_delay -27.52 -24.28 Tighten

designs/sky130hd/aes

Metric Old New Type
finish__timing__setup__ws -0.39 -0.22 Tighten

designs/sky130hd/chameleon

Metric Old New Type
globalroute__antenna_diodes_count 200 126 Tighten
detailedroute__route__wirelength 798366 798308 Tighten

designs/sky130hd/gcd

Metric Old New Type
cts__design__instance__count__setup_buffer 65 42 Tighten
finish__timing__drv__setup_violation_count 76 70 Tighten

designs/sky130hd/ibex

Metric Old New Type
detailedroute__route__wirelength 808993 807636 Tighten
detailedroute__antenna_diodes_count 70 60 Tighten
finish__timing__setup__ws -1.45 -1.02 Tighten
finish__timing__wns_percent_delay -19.68 -17.39 Tighten

designs/sky130hd/jpeg

Metric Old New Type
globalroute__antenna_diodes_count 410 291 Tighten
detailedroute__antenna_diodes_count 94 68 Tighten
finish__timing__setup__ws -0.47 -0.4 Tighten

[WARNING] Multiple clocks not supported. Will use first clock: ext_clk: 15.0000. designs/sky130hd/microwatt

Metric Old New Type
finish__timing__setup__ws -3.1 -2.86 Tighten

designs/sky130hd/riscv32i

Metric Old New Type
finish__timing__setup__ws -1.36 -1.12 Tighten
finish__timing__wns_percent_delay -30.94 -27.63 Tighten

designs/sky130hs/aes

Metric Old New Type
detailedroute__route__wirelength 737983 731288 Tighten
detailedroute__antenna_diodes_count 45 26 Tighten
finish__timing__setup__ws -0.47 -0.17 Tighten
finish__design__instance__area 193310 191253 Tighten

designs/sky130hs/ibex

Metric Old New Type
detailedroute__antenna_diodes_count 38 32 Tighten
finish__timing__setup__ws -0.46 -0.43 Tighten

designs/sky130hs/jpeg

Metric Old New Type
detailedroute__antenna__violating__nets 3 1 Tighten
detailedroute__antenna_diodes_count 810 788 Tighten

designs/sky130hs/riscv32i

Metric Old New Type
globalroute__antenna_diodes_count 3 2 Tighten
finish__timing__setup__ws -0.32 -0.28 Tighten

Messages from CI

[INFO] asap7/cva6 not included in CI. [INFO] asap7/minimal not included in CI. [INFO] asap7/mock-cpu set to not run for this PR. [INFO] asap7/swerv_wrapper set to not run for this PR. [INFO] gf12 not included in the update. [INFO] gf55 not included in the update. [INFO] intel16 not included in the update. [INFO] nangate45/ariane136 set to not run for this PR. [INFO] nangate45/black_parrot set to not run for this PR. [INFO] nangate45/bp_quad not included in CI. [INFO] nangate45/mempool_group not included in CI. [INFO] src not included in the update.

github-actions[bot] avatar Feb 27 '25 20:02 github-actions[bot]

@maliberty FYI, issue has been fixed.

vvbandeira avatar Mar 04 '25 16:03 vvbandeira

@vvbandeira there is now no tightening - is that expected?

maliberty avatar Mar 06 '25 09:03 maliberty

@vvbandeira there is now no tightening - is that expected?

You just missed it. The workflow goes like this:

  1. The action triggers around 8 am UTC, i.e., midnight PST.
  2. If there is a newer version of OR than what is committed to this repo, continue
  3. Update the OR submodule
  4. Force push the update to the branch update-openroad --> Keep only one commit per PR
  5. The force push also removes the old PR description --> Avoid getting polluted by many updates
  6. Add the label UpdateRules
  7. CI will start
  8. When CI finishes, the action UpdateRules starts
  9. If there are rules to be tightened, the bot will commit them and update the PR description
  10. The bot will push the changes, triggering a new CI for sanity check

In summary, every day between Step 5 (~00:05 AM PST) and Step 9 (~2:40 AM PST), the PR description is expected to be empty. Step 10 is expected to finish by 6 am PST daily. We can change the trigger time if you prefer to review the PR at the end of the day instead of the beginning of the day.

You can see in the PR description history when it was updated: image

vvbandeira avatar Mar 06 '25 12:03 vvbandeira

@vvbandeira CI outage

oharboe avatar Apr 09 '25 10:04 oharboe

@vvbandeira CI outage

Likely related to this issue upstream with GH: https://www.githubstatus.com/incidents/mb56b2qv4pyz It is working now.

vvbandeira avatar Apr 09 '25 13:04 vvbandeira

@maliberty merge to pick up https://github.com/The-OpenROAD-Project/OpenROAD/pull/7352 ?

oharboe avatar May 13 '25 08:05 oharboe