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 file #4

Merged
merged 3 commits into from
Oct 16, 2024
Merged

Add new file #4

merged 3 commits into from
Oct 16, 2024

Conversation

newmansc2
Copy link
Contributor

No description provided.

This layout contains a set of loopbacks to characterize the GC_SiN_TE_1550_8degOxide_BB GC.  However, this component has a pin issue.  I believe this is the source of all the DRC issues with my layout.

Running functional verification ("V") on a layout with a single GC_SiN_TE_1550_8degOxide_BB component produces the following error:

Invalid Pin [TOP]
Invalid pin found. Read more about requirements for components: https://github.com/SiEPIC/SiEPIC-Tools/wiki/Component-and-PCell-Layout
polygon: (9.115,0.575;9.115,0.585;9.125,0.585;9.125,0.575)
The components with the pin problem is: GC_SiN_TE_1550_8degOxide_BB
@newmansc2
Copy link
Contributor Author

"Merge branch 'SiEPIC:main' into main"

This commit was an accident. Not sure if it does anything. Please reject my pull request if it is a problem.

moving SiN to Layer 4/0
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).

Please note that we have added a new rule (on May 2, 2024) to SiEPIC-Tools and the EBeam PDK, so please make sure your tools are running the latest version. The rule is: The grating coupler spacing (pitch) must be at least 60.0 microns. This is to help avoid the probe station from accidentally aligning to an adjacent circuit.

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

@lukasc-ubc
Copy link
Member

Applied Nanotools asked us to change to Layer 4/0. I have updated the SiEPIC EBeam PDK.

I updated your design accordingly.

@lukasc-ubc lukasc-ubc merged commit 6a36f54 into SiEPIC:main Oct 16, 2024
2 checks passed
@newmansc2
Copy link
Contributor Author

thank you

@newmansc2
Copy link
Contributor Author

I was not able to update the SiEPIC EBeam PDK using the KLayout package manager. It didn't give me the option to download the latest version (v0.4.20). However, I updated the PDK manually by cloning the repo and copying the files to my KLayout directory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants