-
Notifications
You must be signed in to change notification settings - Fork 64
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
LVS unexpectedly failing #254
Comments
Sorry, pressed wrong button (resulted in first empty problem description). |
Updated the problem description with test case. |
@stafverhaegen-chipflow @atorkmabrains @FaragElsayed2 The output contains some warnings: I have used |
shouldn't the batch-mode deck and GUI-mode deck behave the same? do they share the same implementation? |
@KrzysztofHerman, could you please share here the error message from the batch mode LVS run? |
Here is the output from the command line run:
GUI based run
|
:) what is the exact issue in the batch-mode? |
I tried again with main branch and now for me it does seem to match when running from command line in flat mode; e.g. add |
When I run the C4M deck also in deep mode it also fails. It thus seems that klayout has problems with this layout in deep mode. I don't see immediately something wrong with it. |
Hi @stafverhaegen-chipflow, you have connectivity issues..
to
KLayout LVS should be clean after this update (I would be very surprised if it will not be the case) |
IMHO, from the KLayout LVS reports posted here it's nearly impossible to understand the issue root cause.. |
I was already checking with the change you proposed locally. It does not change anything. I made a new test case: 20241114_lvs_case.tar.gz |
ok, I asked KLayout LVS deck developers to take a look on this |
@stafverhaegen-chipflow Just to confirm my understanding of the issue that you have, there is a test case that pass LVS if you run from GUI and fails if you run from command line.
Could you please elaborate on what I mentioned above? If there are more cases, could you please list them the same way I mentioned above. That would really help us review quickly the root cause. |
Fundamentally, it will fail if deep run and pass if it's flat regardless of the way of invocation. Is that correct? |
One last question, Did you run this test case on other tool (Commercial) and it passed? |
@atorkmabrains, I ran it with commercial tools - there is indeed an LVS issue and I showed how to make it clean |
Indeed |
@sergeiandreyev I presume you ran it hierarchical in the commercial tool, correct? Did you use "Automatch" option in the commercial tool as well to see which exact cell needs changing? I understand that you have provided a change required for the netlist, @stafverhaegen-chipflow Did you test that change and ran LVS in deep mode? |
@atorkmabrains, yes, it was w/ default 'Hierarchical' setting |
@sergeiandreyev That's automatch switch. |
The latest uploaded test case has the change applied. |
Environment
python 3.10, Linux Mint 21, klayout 0.29.5, LVS deck from latest dev branch
Expected Behavior
LVS passing
Actual Behavior
LVS failing
Steps to Reproduce the Problem
One can see that IHP deck reports failing LVS, the PDKMaster generated one reports matching netlists. I also tried with running the IHP deck in flat mode but that did not change anything.
The text was updated successfully, but these errors were encountered: