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.

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

Add new designs #5

Closed
wants to merge 2 commits into from
Closed

Add new designs #5

wants to merge 2 commits into from

Conversation

newmansc2
Copy link
Contributor

I updated the SiEPIC EBeam PDK to v0.4.20 (manually by cloning the repo and copying the files).

SiN still appears to be on layer 5 (if I understand correctly). Please advise.

I updated the SiEPIC EBeam PDK to v0.4.20 (manually by cloning the repo and copying the files).  

SiN still appears to be on layer 5 (if I understand correctly).  Please advise.
@lukasc-ubc
Copy link
Member

Strange. Can you check the EBeam.lyt file has 4/0 for SiN, and restart KLayout?

image

@lukasc-ubc
Copy link
Member

I just updated the PDK in KLayout's Salt Package manager. It may be that you still have the old one installed? Check that the library has the right version number.

image

Changed to 4/0 for one of the ring circuits
Copy link

Welcome, new contributor!

Thank you for uploading your design.

If you have not already checked it, please run the SiEPIC Functional Verification in KLayout, using the menu SiEPIC-Verification-Functional Layout Check (V).

You may continue making updates to your design, or even contributing additonal designs (using a separate file name), until the tape-out deadline.

@newmansc2
Copy link
Contributor Author

It works now. I updated the SiEPIC EBeam PDK to v0.4.21 using the KLayout package manager.

image

image

@newmansc2
Copy link
Contributor Author

thank you

@newmansc2
Copy link
Contributor Author

Closing to fix the files.

@newmansc2 newmansc2 closed this Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants