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

retry adding new designs #120

Merged
merged 9 commits into from
Jul 16, 2024
Merged

retry adding new designs #120

merged 9 commits into from
Jul 16, 2024

Conversation

newmansc2
Copy link
Contributor

DRC issues with the previous submission of these designs should be addressed.

Submit a new version of design from aborted commits.  Uses the GCs and electrical pads from the example EBeam_heaters_BraggGratingwithHeater.gds example.  YAML file is a stub (copy of the example EBeam_heaters_BraggGratingwithHeater.yaml).
Moved the optical text label to the location in the example (EBeam_heaters_BraggGratingwithHeater.gds).
Add DevRec bounding box to custom component.
Move heater segments inside cell MZI_Ring_2_2 in attempt to address DRC issue.
This version should be free from DRC errors (V and D), except for nuisance errors.
This version should be free from DRC errors (V and D), except for nuisance errors.
@lukasc-ubc
Copy link
Member

Running verification on EBeam_heaters_snewman_RMA3.gds
Traceback (most recent call last):
File "/home/runner/work/openEBL-2024-07-Si-Heaters/openEBL-2024-07-Si-Heaters/run_DRC.py", line 95, in
num_errors, errors = check_space_width (layout.top_cell(), rule_table_space_width)
RuntimeError: The layout has multiple top cells in Layout.top_cell

@lukasc-ubc
Copy link
Member

I deleted the extra top cell, and now we are ready to merge!

@lukasc-ubc lukasc-ubc merged commit 2d56b9e into SiEPIC:main Jul 16, 2024
4 checks passed
@newmansc2
Copy link
Contributor Author

@lukasc-ubc thanks very much!

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