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
282 stars 262 forks source link

check antenna violations in CI metrics #2081

Open eder-matheus opened 1 week ago

eder-matheus commented 1 week ago

Fixes https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/issues/1899

eder-matheus commented 1 week ago

@vvbandeira I noticed that to have the check performed when running make metadata, I have to update the metrics with make update_ok for the design. Do I need to do this manually for every design, or do we have something in ORFS that would automatize it?

eder-matheus commented 1 week ago

I added the antenna violation metric to every design available in the public and secure-ci that runs in the secure branches. A few designs need to be included (like asap7/swerv_wrapper). They are not in the pipelines of the secure branches and the designs in tsmc65lp, to which I don't have access.

vvbandeira commented 4 days ago

@vvbandeira I noticed that to have the check performed when running make metadata, I have to update the metrics with make update_ok for the design. Do I need to do this manually for every design, or do we have something in ORFS that would automatize it?

@habibayassin has implemented something in the dashboard, but it only works to update the metrics from the master branch. We will work on adapting the feature to update user's PRs.