Skip to content

Comments

update OR#3024

Merged
maliberty merged 2 commits intoThe-OpenROAD-Project:masterfrom
AcKoucher:update-or
Apr 1, 2025
Merged

update OR#3024
maliberty merged 2 commits intoThe-OpenROAD-Project:masterfrom
AcKoucher:update-or

Conversation

@AcKoucher
Copy link
Contributor

@AcKoucher AcKoucher commented Mar 31, 2025

This should fix the pr-merge/head failure introduced in #3007.

Metrics Update

asap7/aes-block

Metric Old New Type
finish__timing__setup__ws -86.3 -119.26 Failing
finish__timing__drv__hold_violation_count 319 100 Tighten

sky130hd/uW

Metric Old New Type
detailedroute__antenna__violating__nets 0 1 Failing

Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>
@AcKoucher AcKoucher requested a review from maliberty March 31, 2025 15:13
@AcKoucher
Copy link
Contributor Author

I'm also running a Secure-CI.

@maliberty
Copy link
Member

Nothing to review yet - please ping when you have the metrics updates in.

asap7/aes-block:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| finish__timing__setup__ws                     |    -86.3 |  -119.26 | Failing  |
| finish__timing__drv__hold_violation_count     |      319 |      100 | Tighten  |

sky130hd/uW:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| detailedroute__antenna__violating__nets       |        0 |        1 | Failing  |

Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>
@AcKoucher
Copy link
Contributor Author

AcKoucher commented Apr 1, 2025

@maliberty
The public pipeline had a couple of needed updates which I included here. Secure is green.

@maliberty maliberty merged commit c7a4e7e into The-OpenROAD-Project:master Apr 1, 2025
7 checks passed
@AcKoucher AcKoucher deleted the update-or branch April 1, 2025 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants