Add SiEPIC EBeam PDK and fix chip base placement#29
Merged
Conversation
Contributor
Author
|
Tested with example gds from the SiEPIC repo. 
|
dnltz
reviewed
Apr 29, 2026
Contributor
|
@tomflexcompute LGTM! There is one remaining issue. Can you please add a signed-off-by to your commits? |
- New SiEPIC EBeam PDK (silicon photonics, 220 nm SOI):
* configs/siepic.yaml — 15-layer stack (Si, SiRib90, SiLitho193, SiN,
N/N++ doping, OxideOpen, M1Heater, VC, M2Router, MOpen, three trenches),
derived from the upstream EBeam.lyp layer numbers.
* configs/colors/siepic/{realistic,fancy,marketing,marketing-gold}.yaml
* Registered in PDK_CONFIGS and the pre-import dialog enum.
* README updated.
- Fix chip base / camera placement for layouts whose bounding box
doesn't start at the GDS origin. setup_chip_scene() now takes the
real (x_min, y_min, x_max, y_max) corners instead of width/height,
and the importer passes the true bbox in the uncropped path and
(0,0)..(crop_w,crop_h) in the cropped path (polygons get shifted
to the origin in that case).
Signed-off-by: tomflexcompute <tom@flexcompute.com>
Revised copyright owner Signed-off-by: tomflexcompute <tom@flexcompute.com>
Contributor
Author
Thanks! Should be all good. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
New SiEPIC EBeam PDK (silicon photonics, 220 nm SOI) (Add SiEPIC Ebeam PDK #14):
Fix chip base / camera placement for layouts whose bounding box doesn't start at the GDS origin. setup_chip_scene() now takes the real (x_min, y_min, x_max, y_max) corners instead of width/height, and the importer passes the true bbox in the uncropped path and (0,0)..(crop_w,crop_h) in the cropped path (polygons get shifted to the origin in that case).
Closes #14