-
Notifications
You must be signed in to change notification settings - Fork 385
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
Rework Placement #1684
Merged
Merged
Rework Placement #1684
Conversation
This file contains 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
+ Add warning when attempting to run Sylvain's script without `DPL_CELL_PADDING` >= 0 + Add warning when DPL_CELL_PADDING is set to greater than GPL_CELL_PADDING ~ `PL_TARGET_DENSITY` incorporates value of `GPL_CELL_PADDING` in calculation ~ Updated various config files to have an explicit padding of 2
Why not change the default of |
@marwaneltoukhy It triples the design area and is only needed for the custom script, which is disabled by default in favor of DIS 3. |
marwaneltoukhy
approved these changes
Mar 22, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
DPL_CELL_PADDING
>= 0~ Use $HOME/.volare as the default PDK location
~
PL_TARGET_DENSITY
incorporates value ofGPL_CELL_PADDING
in calculation~ Updated various config files to handle new default padding options
~ CI bugfixes + support for Volare 0.7.1
[ci ets]
Depends on RTimothyEdwards/open_pdks#358.