Peter

@maliberty the message is just there to indicate that nothing was generated in a particular grid. No vias, straps, rings, etc. So the powergrid is incomplete. I'll look at the...

@maliberty There is a missing: ``` add_pdn_connect -grid stdcell_grid -layers "met4 met5" ```

It should be added: https://github.com/The-OpenROAD-Project/OpenLane/blob/3bc9d02d0b34ad032921553e512fbe4bebf1d833/scripts/openroad/common/pdn_cfg.tcl#L126 Without it there are no vias added to the macro

Couldn't this be done like the git describe thing and add a string to the version header file?

@maliberty you could do something like yosys: https://github.com/YosysHQ/yosys/blob/91fbd58980e87ad5dc0a5d37c049ffaf5ab243dd/Makefile#L770 Have cmake build the string and write that into a file like it does with the version.h header.

How come the pins were placed overlapping the power straps? That seems to be the actual issue here based on my read of this.

@oharboe that doesn't explain why PPL put in the pins on top of the straps

If the pins are marked FIXED then PDN will avoid them, if they are just PLACED then PDN is expecting a second call to the pin placer tool (PPL) to...

@oharboe sure, I don't disagree (I just think the implementation in the PR is not quite consistent with the rest of the codebase), I generally don't like that PDN places...

@maliberty Ill take a look tomorrow if I can