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.

Sign up for GitHub

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

Jump to bottom

My design (MathieuG38 on Edx) #63

Merged
merged 6 commits into from
Oct 13, 2024
Merged

My design (MathieuG38 on Edx) #63

merged 6 commits into from
Oct 13, 2024

Conversation

Matman42
Copy link
Contributor

@Matman42 Matman42 commented Oct 2, 2024

No description provided.

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 2, 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).

Please note that we have added a new rule (on May 2, 2024) to SiEPIC-Tools and the EBeam PDK, so please make sure your tools are running the latest version. The rule is: The grating coupler spacing (pitch) must be at least 60.0 microns. This is to help avoid the probe station from accidentally aligning to an adjacent circuit.

You may continue making updates to your design, or even contributing additonal designs (using a separate file name), until the tape-out deadline.

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 2, 2024

Thank you for your pull request! 👋

@omidesml
Copy link
Contributor

omidesml commented Oct 2, 2024

Would you please delete the laser and detectors from the layout.
There is a problem with labels. make sure they are unique
Clean the layout and update the pull request

@Matman42
Updated design
8bf92a6 
I removed the lasers and detectors from the layout
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 2, 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).

Please note that we have added a new rule (on May 2, 2024) to SiEPIC-Tools and the EBeam PDK, so please make sure your tools are running the latest version. The rule is: The grating coupler spacing (pitch) must be at least 60.0 microns. This is to help avoid the probe station from accidentally aligning to an adjacent circuit.

You may continue making updates to your design, or even contributing additonal designs (using a separate file name), until the tape-out deadline.

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 2, 2024

Thank you for your pull request! Staff will review your submission, and if it passes all checks, it will be merged. After merging, you may check the merged file. 👋

@Matman42
Copy link
Contributor Author

Matman42 commented Oct 2, 2024

Hi, I removed the lasers and detectors and all labels seem unique

@lukasc-ubc
Copy link
Member

  • Found overlapping components: ebeam_wg_integral_1550, ebeam_wg_integral_1550
  • Found overlapping components: ebeam_wg_integral_1550, ebeam_wg_integral_1550
  • Found overlapping components: ebeam_gc_te1550, ebeam_gc_te1550
  • Found overlapping components: ebeam_y_1550, ebeam_y_1550
  • Found overlapping components: ebeam_gc_te1550, ebeam_gc_te1550
  • Found overlapping components: ebeam_gc_te1550, ebeam_gc_te1550
  • Found overlapping components: ebeam_bdc_te1550, ebeam_bdc_te1550
  • Found overlapping components: ebeam_wg_integral_1550, ebeam_wg_integral_1550
  • Found overlapping components: ebeam_wg_integral_1550, ebeam_wg_integral_1550
  • Found overlapping components: ebeam_wg_integral_1550, ebeam_wg_strip_1550
  • Found overlapping components: ebeam_wg_integral_1550, ebeam_wg_strip_1550
    21 layout errors detected.
    Please review errors using the 'Marker Database Browser'.
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: Grating coupler: minimum spacing: The grating coupler spacing (pitch) must be at least 60.0 microns.
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: Grating coupler: minimum spacing: The grating coupler spacing (pitch) must be at least 60.0 microns.
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: Grating coupler: minimum spacing: The grating coupler spacing (pitch) must be at least 60.0 microns.
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: Grating coupler: minimum spacing: The grating coupler spacing (pitch) must be at least 60.0 microns.
    Error: Grating coupler: minimum spacing: The grating coupler spacing (pitch) must be at least 60.0 microns.
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: Overlapping component: Overlapping components (defined as overlapping DevRec layers; touching is ok)
    Error: opt_in label: same: Automated test opt_in labels should be unique.
    Error: Fibre array configuration: Circuit must be connected such that there is at most 1 Grating Coupler(s) above the opt_in label (laser injection port) and at most 2 Grating Coupler(s) below the opt_in label.
    Grating couplers must be on a 127.0 micron pitch, vertically arranged.
    Error: Fibre array configuration: Circuit must be connected such that there is at most 1 Grating Coupler(s) above the opt_in label (laser injection port) and at most 2 Grating Coupler(s) below the opt_in label.
    Grating couplers must be on a 127.0 micron pitch, vertically arranged.
    Error: Fibre array configuration: Circuit must be connected such that there is at most 1 Grating Coupler(s) above the opt_in label (laser injection port) and at most 2 Grating Coupler(s) below the opt_in label.
    Grating couplers must be on a 127.0 micron pitch, vertically arranged.
    Error: Fibre array configuration: Circuit must be connected such that there is at most 1 Grating Coupler(s) above the opt_in label (laser injection port) and at most 2 Grating Coupler(s) below the opt_in label.
    Grating couplers must be on a 127.0 micron pitch, vertically arranged.
  • Previous label: ('SiEPIC-Tools verification: 34 errors\n2024-10-01 15:16:44\nSiEPIC-Tools v0.5.11\ntechnology: EBeam\nwin32\nPython: 3.11.6 (main, Oct 3 2023, 09:12:56) [GCC 13.2.0 64 bit (AMD64)], C:\Users\skywa\AppData\Roaming\KLayout\pymod\n['0', '29', '6']',r0 -142920,-119250) s=100
  • Previous label: ('SiEPIC-Tools verification: 6 errors\n2024-10-01 15:20:49\nSiEPIC-Tools v0.5.11\ntechnology: EBeam\nwin32\nPython: 3.11.6 (main, Oct 3 2023, 09:12:56) [GCC 13.2.0 64 bit (AMD64)], C:\Users\skywa\AppData\Roaming\KLayout\pymod\n['0', '29', '6']',r0 0,0) s=100
  • Previous label: ('SiEPIC-Tools verification: 6 errors\n2024-10-01 15:28:08\nSiEPIC-Tools v0.5.11\ntechnology: EBeam\nwin32\nPython: 3.11.6 (main, Oct 3 2023, 09:12:56) [GCC 13.2.0 64 bit (AMD64)], C:\Users\skywa\AppData\Roaming\KLayout\pymod\n['0', '29', '6']',r0 0,0) s=100
    KLayout SiEPIC-Tools version 0.5.12
    21
    Errors detected: EBeam_MathieuG38_A.gds, 21 errors.

@lukasc-ubc lukasc-ubc added the bug Something isn't working label Oct 9, 2024
@Matman42
My final design overwritten
d3bf6a3 
Please waive the two errors, they are related to a beta instance
@github-actions GitHub Actions
Copy link

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).

Please note that we have added a new rule (on May 2, 2024) to SiEPIC-Tools and the EBeam PDK, so please make sure your tools are running the latest version. The rule is: The grating coupler spacing (pitch) must be at least 60.0 microns. This is to help avoid the probe station from accidentally aligning to an adjacent circuit.

You may continue making updates to your design, or even contributing additonal designs (using a separate file name), until the tape-out deadline.

@github-actions GitHub Actions
Copy link

Thank you for your pull request! 👋

@Matman42
My final design overwritten
72239f0 
Changed the spiral for a spiral paperclip to lift the error
@github-actions GitHub Actions
Copy link

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).

Please note that we have added a new rule (on May 2, 2024) to SiEPIC-Tools and the EBeam PDK, so please make sure your tools are running the latest version. The rule is: The grating coupler spacing (pitch) must be at least 60.0 microns. This is to help avoid the probe station from accidentally aligning to an adjacent circuit.

You may continue making updates to your design, or even contributing additonal designs (using a separate file name), until the tape-out deadline.

@github-actions GitHub Actions
Copy link

Thank you for your pull request! 👋

@github-actions GitHub Actions
Copy link

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).

Please note that we have added a new rule (on May 2, 2024) to SiEPIC-Tools and the EBeam PDK, so please make sure your tools are running the latest version. The rule is: The grating coupler spacing (pitch) must be at least 60.0 microns. This is to help avoid the probe station from accidentally aligning to an adjacent circuit.

You may continue making updates to your design, or even contributing additonal designs (using a separate file name), until the tape-out deadline.

@github-actions GitHub Actions
Copy link

Thank you for your pull request! 👋

@github-actions GitHub Actions
Copy link

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).

Please note that we have added a new rule (on May 2, 2024) to SiEPIC-Tools and the EBeam PDK, so please make sure your tools are running the latest version. The rule is: The grating coupler spacing (pitch) must be at least 60.0 microns. This is to help avoid the probe station from accidentally aligning to an adjacent circuit.

You may continue making updates to your design, or even contributing additonal designs (using a separate file name), until the tape-out deadline.

@github-actions GitHub Actions
Copy link

Thank you for your pull request! 👋

@Matman42
Copy link
Contributor Author

I did a last minute change to replace the spiral with the spiral paperclip to lift the remaining errors.

@lukasc-ubc lukasc-ubc merged commit bd09a6a into SiEPIC:main Oct 13, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Matman42 @omidesml @lukasc-ubc