Fixed CTS_CLUSTER_SIZE & CTS_CLUSTER_DIAMETER handling#3122
Closed
jeffng-or wants to merge 2 commits intoThe-OpenROAD-Project:masterfrom
Closed
Fixed CTS_CLUSTER_SIZE & CTS_CLUSTER_DIAMETER handling#3122jeffng-or wants to merge 2 commits intoThe-OpenROAD-Project:masterfrom
jeffng-or wants to merge 2 commits intoThe-OpenROAD-Project:masterfrom
Conversation
jeffng-or
force-pushed
the
cts-fix-cluster-vars
branch
from
93de955 to
b0b0507
Compare
Contributor
Author
|
The failures are due to metadata changes as a result of fixing the CTS cluster variables. @maliberty , @vvbandeira , Is it OK to set the UpdateRules label and get this change merged? |
Contributor
Author
|
Looks like the failures for both pr-head and pr-merge are all rules updates. Setting the UpdateRules label and hoping that it goes through. Looks like it needs a review approval though. |
maliberty
approved these changes
May 6, 2025
updated sky130hd microwatt CTS cluster variables updated rules Signed-off-by: Jeff Ng <jeffng@precisioninno.com>
jeffng-or
force-pushed
the
cts-fix-cluster-vars
branch
from
b0b0507 to
ab3be9a
Compare
Contributor
Author
|
I went ahead and ran the rules files for the failing tests. Here are the updates: gf180_ibex
gf180_jpeg
gf180_riscv32i
nangate45_swerv
sky130hd_microwatt
sky130hd_riscv32i
sky130hs_ibex
sky130hs_riscv32i
|
Signed-off-by: Jeff Ng <jeffng@precisioninno.com>
Contributor
Author
|
Replaced with #3142 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.