Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Integrate KLayout DRC for sky130 #2059

Merged
merged 14 commits into from
Dec 11, 2023
Merged

Integrate KLayout DRC for sky130 #2059

merged 14 commits into from
Dec 11, 2023

Conversation

kareefardi
Copy link
Collaborator

@kareefardi kareefardi commented Nov 30, 2023 •
edited
Loading

~ Bump open_pdks version to cd1748bb197f9b7af62a54507de6624e30363943
~ Rename PRIMARY_SIGNOFF_TOOL to PRIMARY_GDSII_STREAMOUT_TOOL
~ Update documentation of RUN_MAGIC_DRC and RUN_KLAYOUT_DRC
+ Integrate KLayout DRC for sky130
+ Add the following variables:

KLAYOUT_DRC_THREADS
QUIT_ON_KLAYOUT_DRC
RUN_KLAYOUT_DRC
KLAYOUT_DRC_RUNSET - PDK variable
KLAYOUT_DRC_OPTIONS - PDK variable

- Remove KLAYOUT_DRC_KLAYOUT_GDS

@kareefardi kareefardi changed the title initial klayout drc integration Integrate KLayout DRC for sky130 Dec 6, 2023
@kareefardi kareefardi marked this pull request as ready for review December 7, 2023 09:40
@kareefardi kareefardi requested a review from donn December 7, 2023 09:40
donn
donn previously approved these changes Dec 11, 2023
View reviewed changes
@donn donn self-requested a review December 11, 2023 13:00
donn
donn previously approved these changes Dec 11, 2023
View reviewed changes
@donn donn merged commit 571581d into master Dec 11, 2023
84 checks passed
@donn donn deleted the klayout-drc branch December 11, 2023 15:31
@kareefardi kareefardi mentioned this pull request Dec 24, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@donn donn donn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kareefardi @donn