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

Crash, and problem #9

Open
lukasc-ubc opened this issue May 26, 2024 · 9 comments
Open

Crash, and problem #9

lukasc-ubc opened this issue May 26, 2024 · 9 comments

Comments

@lukasc-ubc
Copy link
Member

@JymBone5

There is a problem with the examples provided and they don't work with the current version of the tool.

SiEPIC/openEBL-2024-07-Si-Heaters#3

image

I tried loading the file EBeam_heaters_BraggGratingwithHeater.yaml, and it caused the testcreator to crash. Can you please add error checking?

I tried using the .txt file extracted from KLayout, and I also get the same two errors:
image

thank you

@boy4429856821
Copy link

boy4429856821 commented Jun 5, 2024

Is there any update on this? @lukasc-ubc I'm unsure I can submit my design without the yaml creation tool... I could make it by scratch but doesn't sound fun.... Do you have any suggestions?

@lukasc-ubc
Copy link
Member Author

@JymBone5, progress report please?

@PetervandenDoel
Copy link
Contributor

I only have access to Ida this week, I will discuss the bug with John and fix the issue next week

@PetervandenDoel
Copy link
Contributor

@boy4429856821 Can you try labelling your electrical pad names without the _G at the end?

@boy4429856821
Copy link

boy4429856821 commented Jun 12, 2024

Hi @PetervandenDoel

Yes, I removed the _G at the end and see the same error:
image

I am using the following code to launch the GUI:
image

Here is the coordinates .txt file as referrence:
Ebeam_Heaters_ColemanTheSoulman.txt

@PetervandenDoel
Copy link
Contributor

PetervandenDoel commented Jun 12, 2024

@boy4429856821

It seems that there has been some confusion about the convention of how to label electrical pads. In the README for SiEPIC_testcreator it says that you should label only the top pad and label it with elec_ followed by the device ID. We used to use a convention of labelling one pin as _G for ground and one pin as _S for source, but our group is abandoning that convention. It seems our documentation has not been fully updated and the electrical automation software is still being worked on. Please follow the instructions on the SiEPIC_testcreator readme, relabel your electrical pads, and let me know if it works.

Thanks for your patience

-Peter

@boy4429856821
Copy link

@PetervandenDoel
Hi, your suggestion of only labelling the top pad and label it with elec_ followed by device ID did not work. Please provide me with a coordinate .txt file that you know that works and I can try it...

Coleman

@PetervandenDoel
Copy link
Contributor

@boy4429856821
Hello,

What I did was I took a default layout
EBeam_heaters_BraggGratingwithHeater.gds

And I simply removed the _G from the end and i generated the .txt file from there. It throws an error with regards to the last line which has to do with an unimplemented photonic wire bonding feature, but I was able to generate a proper yaml file from that

@boy4429856821
Copy link

boy4429856821 commented Jun 15, 2024

@PetervandenDoel

Alright I was able to figure it out....:

1.) I was using "opt_in" instead of "opt" in the optical label
2.) I was unable to put my name into the optical label... I can probably get away with that by including it in the device name...
3.) OneDrive was causing some weird behavior...

image

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

No branches or pull requests

3 participants