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

Final updated files #200

Merged
merged 1 commit into from
May 19, 2024
Merged

Final updated files #200

merged 1 commit into from
May 19, 2024

Conversation

SoibamArunaChanu
Copy link
Contributor

No description provided.

@lukasc-ubc lukasc-ubc merged commit 162e623 into SiEPIC:main May 19, 2024
2 checks passed
@lukasc-ubc
Copy link
Member

@SoibamArunaChanu

Unfortunately, several of your designs have a big problem (which has an easy fix). Namely, you have changed the cell names of the grating couplers. As described in the Readme file, the cell names with _BB cannot be changed, otherwise the replacement script does not recognize them. You changed their names to PhCxxxx.

So as is, your design won't function. Can you please fix asap?

The designs that have this problem are:
PBS1
PBS3
PBS5

image

@SoibamArunaChanu
Copy link
Contributor Author

@lukasc-ubc Thank you for pointing out the problem.
I am not quite sure how I can fix this since it is not shown here.
gc_error

I have deleted the grating couplers and added them again. Will this fix the problem?

@lukasc-ubc
Copy link
Member

let me check.

@lukasc-ubc
Copy link
Member

I'm not sure where the PhC14$1 is coming from. But indeed, the file you uploaded doesn't seem to have it! Strange... I am investigating and will try to fix it.

@lukasc-ubc
Copy link
Member

strange bug in KLayout? KLayout/klayout#1714

@SoibamArunaChanu
Copy link
Contributor Author

I found out that, I have used the name "PhCXX" to save my .gds files exported from Lumerical FDTD.

I am not sure why some of the layouts are having this problem, and some are not. I am freshly designing the ones with the problem.

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