-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat!: quit on hold violations on all corners #587
base: dev
Are you sure you want to change the base?
Conversation
Signed-off-by: Kareem Farid <[email protected]>
Signed-off-by: Kareem Farid <[email protected]>
Metric comparisons are in beta. Please report bugs under the issues tab.
Full tables ► https://gist.github.com/openlane-bot/39c1dff69b643845ec8ce2a0db2f7388 |
Signed-off-by: Kareem Farid <[email protected]>
Signed-off-by: Kareem Farid <[email protected]>
Signed-off-by: Kareem Farid <[email protected]>
This reverts commit 7c0a979.
While you're here, tt needs to be taken from the PDK instead of being set as part of the variable name. This already caused an issue or two with some of our internal PDKs. |
Steps
Checker.HoldViolations
HOLD_VIOLATION_CORNERS
to['*']
causing erroron hold violations on all corners
Depends on efabless/openlane2-ci-designs#35
BREAKING CHANGE: Designs with hold violations on other corners than
*tt*
will now need updatingFixes #523