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

CryoChip #12

Merged
merged 39 commits into from
Nov 25, 2024
Merged

CryoChip #12

merged 39 commits into from
Nov 25, 2024

Conversation

joerdsonsilva
Copy link
Contributor

No description provided.

@britovski
Copy link

@KrzysztofHerman We are tracking the issue opened... since that is a bug, we can consider that is ready for tape-out? We have any room for gds modification or we need to freeze?

@KrzysztofHerman
Copy link
Contributor

We are reviewing the submissions. I will be back with our observations here. Anyway there is still a headroom for improvements.

@KrzysztofHerman
Copy link
Contributor

@joerdsonsilva

After the revision we would like to ask you to provide some documentation and background of your project,

Although the design is DRC clean it have a lot of LVS issues, are you aware of that ? Will it be fixed ?

@joerdsonsilva
Copy link
Contributor Author

The project documentation has been added. Yes, the LVS issues will be fixed. The LVS before the Fill is clean, but the LVS after the Fill is showing some errors. An issue will be opened regarding this matter. We have any room for gds modification?

@KrzysztofHerman
Copy link
Contributor

@joerdsonsilva @britovski please update your GDS fixing contact sizes in the last row (3-rd and 4-th) column. Now it is too big. The correct size is 0.16x0.16 um

@KrzysztofHerman
Copy link
Contributor

image

@joerdsonsilva
Copy link
Contributor Author

joerdsonsilva commented Nov 22, 2024

Updated as requested. The errors presented in the PR DRC are from AFil.g2 and M1-5Fil.h, is this normal behavior at this stage?

@KrzysztofHerman
Copy link
Contributor

@joerdsonsilva @britovski

Congratulations your design was accepted for production
File name: FMD_QNC_CryoChip_4.gds
Top cell name: CryoChip
Testfield: T576

I had to remove the logo because of critical DRC issues.

@KrzysztofHerman KrzysztofHerman merged commit 873382a into IHP-GmbH:main Nov 25, 2024
1 check failed
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.

3 participants