I'm getting weird PDN stuff with my project
```INF...
# openlanem
Matt Venn
03/16/2022, 7:52 PMI'm getting weird PDN stuff with my project
Copy code
INFO PSM-0064] Number of voltage sources = 88.
[WARNING PSM-0038] Unconnected PDN node on net vccd1 at location (367.200um, 1910.580um), layer: 6.
[WARNING PSM-0038] Unconnected PDN node on net vccd1 at location (367.200um, 1910.000um), layer: 5.
[ERROR]: PDN generation failed.Matt Venn
03/16/2022, 7:53 PMI also have this
Matt Venn
03/16/2022, 7:53 PMrepo is here - should fail fast with make user_project_wrapper
Matt Venn
03/16/2022, 7:53 PMMatt Venn
03/16/2022, 7:53 PMany thoughts @User @User? should I try to make the reproducable and open an issue?
d
donn
03/16/2022, 7:54 PMdefinitely yeah
m
Matt Venn
03/16/2022, 7:57 PMit fails after PDN, so I add the -1 to pdn.tcl?
d
donn
03/16/2022, 7:57 PMyou don't need to add the -1: if it outright failed, it should've packaged it automatically
m
Matt Venn
03/16/2022, 7:58 PMno it didn't
d
donn
03/16/2022, 7:59 PMalright, can we connect tomorrow? i've had a long day
m
Matt Liberty
03/16/2022, 8:00 PMFYI, pdngen is in the middle of a rewrite from tcl to c++ which I hope will land next week.
Matt Liberty
03/16/2022, 8:01 PM@User in your "I also have this" it is likely to be a repair strap in a channel between macros. If not that would be of interest
m
Matt Venn
03/16/2022, 8:02 PMok
Matt Venn
03/16/2022, 8:02 PMI have the reproducible. @User what do you prefer?
Matt Venn
03/16/2022, 8:02 PM@User - sure no worries
m
Matt Liberty
03/16/2022, 8:03 PM@User Please open an issue at attach it
m
Matt Venn
03/16/2022, 8:03 PMok
Matt Venn
03/16/2022, 8:03 PMin openlane?
Matt Venn
03/16/2022, 8:03 PMor openroad
m
Matt Liberty
03/16/2022, 8:03 PMOR is fine with a standalone reproducer
m
Matt Venn
03/16/2022, 8:09 PM@User feedback welcomed on that issue: https://github.com/The-OpenROAD-Project/OpenROAD/issues/1703
Matt Venn
03/16/2022, 8:09 PMtrain me to do it well so I can improve the quality of your issues
m
Matt Liberty
03/16/2022, 8:33 PMThe test case looks fine. The new c++ version doesn't have this issue but it does have a different one 🤷
3 Views