netgen icon indicating copy to clipboard operation
netgen copied to clipboard

Latest netgen tool hanging

Open dineshannayya opened this issue 2 years ago • 5 comments

I see latest netgen tool hanging after below messages

Circuits match with 11 symmetries.
Resolving automorphisms by property value.
Resolving automorphisms by pin name.
Netlists match with 9 symmetries.

Last working netgen checkout is : 7ee50a3f8fc41dffad74adec7d32a691586351cf commit 7ee50a3f8fc41dffad74adec7d32a691586351cf Author: Tim Edwards [email protected] Date: Mon Mar 1 16:33:55 2021 -0500

Attached the example directory Run command: run_cmd

exp6.tar.gz

dineshannayya avatar Aug 08 '21 10:08 dineshannayya

I'm not sure what you mean by "latest netgen". The commit you referenced is dated March 1, which is 30 commits behind the current version.

Nevertheless, the example does demonstrate the issue on the current version of netgen. I have never seen an example before where the symmetry breaking failed to reduce the number of symmetries. I will need to investigate. Thanks for preparing the repeatable test case.

RTimothyEdwards avatar Aug 09 '21 01:08 RTimothyEdwards

I mean, last working netgen version for this example is checkout: 7ee50a3f8fc41dffad74adec7d32a691586351cf

dineshannayya avatar Aug 09 '21 05:08 dineshannayya

Sorry, I misunderstood. That makes perfect sense. On the next commit following 7ee50a3, I removed the "fast symmetry breaking" method and replaced it with something else. The fast symmetry breaking worked well for synthesized digital circuits, but it was not guaranteed to work correctly. Since it is in principle impossible for the new symmetry breaking method to do what it's doing in your example, it is presumably an unexpected bug, and hopefully a simple one.

RTimothyEdwards avatar Aug 09 '21 13:08 RTimothyEdwards

Netgen is not hung; it is just taking a very, very long time to break symmetry because the standard cells were not allowed to combine in parallel. Your underlying problem is that you do not have the most recent version of open_pdks and the corresponding netgen setup file, which contains the code that allows parallelization of standard cells.

RTimothyEdwards avatar Aug 09 '21 16:08 RTimothyEdwards

Hi Timothy

I have tried with latest OpenPDK :

commit 6de165cc41b9f1050006d37288c8121c23756406 (HEAD -> master, tag: 1.0.205, origin/open_pdks-1.0, origin/master, origin/HEAD) Author: Tim Edwards @.*> Date: Sun Aug 8 20:45:43 2021 -0400

Modified VERSION to go along with the merge of pull request #142
from Max Chen.

With Netgen version:

commit e773739e7d41fb645a93069624c7685115f52451 (HEAD -> master, origin/master, origin/HEAD) Author: Tim Edwards @.*> Date: Fri Aug 6 11:37:29 2021 -0400

Updated VERSION as a forced change to check continuous integration
on github.

In the Run, I see there are series of format issue

Magic 8.2 revision 157 - Compiled on Thu, 07 Nov 2019 21:14:45 +0100. Starting magic under Tcl interpreter Using the terminal as the console. Using NULL graphics device. Processing system .magicrc file Sourcing design .magicrc for technology sky130A ... 2 Magic internal units = 1 Lambda /home/dinesha/workarea/opencore/pdk/sky130A/libs.tech/magic/sky130A.tech: line 19: section version: Badly formed version line Usage: {version text}|{description text} /home/dinesha/workarea/opencore/pdk/sky130A/libs.tech/magic/sky130A.tech: line 608: section compose: Unrecognized layer (type) name "obsli,obsm1" /home/dinesha/workarea/opencore/pdk/sky130A/libs.tech/magic/sky130A.tech: line 796: section cifoutput: Unknown statement "bridge". /home/dinesha/workarea/opencore/pdk/sky130A/libs.tech/magic/sky130A.tech: line 802: section cifoutput: Unknown statement "close". /home/dinesha/workarea/opencore/pdk/sky130A/libs.tech/magic/sky130A.tech: line 803: section cifoutput:

I have attached the example and sky130A.tech for your reference.

Not sure if there is any setup issue. My older netgen setup still works.

-Dinesh.A

On Mon, Aug 9, 2021 at 9:56 PM R. Timothy Edwards @.***> wrote:

Netgen is not hung; it is just taking a very, very long time to break symmetry because the standard cells were not allowed to combine in parallel. Your underlying problem is that you do not have the most recent version of open_pdks and the corresponding netgen setup file, which contains the code that allows parallelization of standard cells.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/RTimothyEdwards/netgen/issues/28#issuecomment-895362995, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB7HAVX6A264RD4AQKAXCP3T376RVANCNFSM5BYKSQTQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

dineshannayya avatar Aug 14 '21 09:08 dineshannayya