Kareem Farid

### Prompt `DIODE_INSERTION_STRATEGY` `4` (and possibly `5`) isn't using padding specified by `DIODE_PADDING` during legalizing inserted diodes ### Proposal All strategies should respect `DIODE_PADDING`

### Prompt `-skip_trim` is a pdngen flag: `Skip the metal trim step, which attempts to remove metal stubs`. it was needed to duplicate a pdn in a local design. ###...

### Prompt When `QUIT_ON_LVS_ERROR`, openlane doesn't report any lvs errors at the end of the flow. ### Proposal The intention behind the variable (if not then it should be) is...

### Prompt In `cts.tcl` `SYNTH_MAX_TRAN` is being converted to seconds by multiplying by `1e-9` (See: https://github.com/The-OpenROAD-Project/OpenLane/blob/63b5966c2874da8bf01a831f0ee9dc68382a4958/scripts/openroad/cts.tcl#L37). That got me thinking that `SYNTH_MAX_TRAN` is assumed to be in nanoseconds. However, it's...

`RUN_LVS` is defined in `configuration/lvs.tcl` Signed-off-by: Kareem Farid

### Prompt Instead of manually removing buffer on specific nets utilize the recently added `set_dont_touch` from openROAD. ### Proposal Instead of manually removing buffer on specific nets utilize the recently...

they wrap around `mgmt_core_wrapper` to connect to `mgmt_protect`

GF180MCU `USEMINSPACING`

1

Again I am not sure if this is the right place. The tech lef for gf180mcu has `USEMINSPACING` set to `ON` which is in my opinion mostly dangerous. Setting it...

GF180MCU tech lef SpacingRange not supported in OpenROAD

2

While this is a liability on the tool, I believe that it is within the capabilities of open_pdks to customize source files of a pdk.

Sky130 ff libs missing table templates

5

### Description STA warns about missing template. Reading ff libs bombards the logs with these warnings ``` table template ccsn_ovrf not found. table template ccsn_ovrf not found. table template ccsn_ovrf...