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

removed DC test structures, removed MM test structures, fixed errors #62

Closed
wants to merge 6 commits into from

Conversation

DantePrins
Copy link
Contributor

Regarding space:
There are 6 files now, if the FaML and FAVE ring8001100 made it onto the chip that would be great, the rest are bracketing.
My fault for not submitting before yesterday.

I removed the DC test structures.
I removed the MM test structures.

errors

I think the multi-mode waveguides test structures were corrupted, they looked fine on my end. I removed them regardless, we can do the measurement on a future chip.

I added floorplans to the submitted files.

@lukasc-ubc
Copy link
Member

I already have merged your previous one. I'd rather not merge this, as it changes the FloorPlan. I added a new rule in the DFT file specifically for the FAVE couplers, which I implemented via my merge script on the files you submitted already.

Download the merged file, and see if it ok as is.

Thanks @DantePrins !

@lukasc-ubc lukasc-ubc closed this Nov 6, 2024
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