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

SiN mask design files-hkalagara #51

Merged
merged 4 commits into from
Nov 5, 2024
Merged

SiN mask design files-hkalagara #51

merged 4 commits into from
Nov 5, 2024

Conversation

hkalagara
Copy link
Contributor

consists of MZIs, Rings, Waveguides of different widths

consists of MZIs, Rings, Waveguides of different widths
Copy link

github-actions bot commented Nov 3, 2024

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.

@lukasc-ubc
Copy link
Member

We are very tight on space, so am I not sure if we will be able to fit all the submissions. I will do my best.

@lukasc-ubc
Copy link
Member

I'm sorry, but this design won't work. It contains Silicon (this is only a SiN run), and also the old 1/5 layer.

Please update your PDK and tools to fix this.

The grating couplers are also not the correct ones. Please read the DFT.md file.

@lukasc-ubc lukasc-ubc added the bug Something isn't working label Nov 4, 2024
replaced the Grating couplers, SiN waveguides
@hkalagara
Copy link
Contributor Author

Uploaded the files with corrections. Okay to skip my designs if no space. Made them only for practice.

@lukasc-ubc lukasc-ubc merged commit 93de9f5 into SiEPIC:main Nov 5, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working greeted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants