OpenROAD icon indicating copy to clipboard operation
OpenROAD copied to clipboard

update to initialize voltage_domains if not set in the config

Open openroad-robot opened this issue 4 years ago • 4 comments

Signed-off-by: ibrahimkhairy [email protected]

This PR has the following two changes for pdngen:

  1. pdngen will initialize voltage_domains if not set in the config
  2. pdngen will return a detailed error message when strap pitch exceeds design width or height

openroad-robot avatar Sep 03 '21 13:09 openroad-robot

@Colin-Holehouse this is waiting on your review

maliberty avatar Oct 12 '21 19:10 maliberty

@Colin-Holehouse this is now months old. Please review so we can close this.

maliberty avatar Nov 29 '21 16:11 maliberty

@arlpetergadfort is there anything here that should be rolled into your c++ version?

maliberty avatar Feb 14 '22 15:02 maliberty

@arlpetergadfort is there anything here that should be rolled into your c++ version?

@maliberty I think it's covered by the determinePowerGroundNets and the automatic domain creation. The stuff added in ICeWall will likely break, but I don't think it's needed anyways.

gadfort avatar Feb 14 '22 15:02 gadfort

obsolete

maliberty avatar Apr 17 '23 20:04 maliberty