FYI those who are using bipolar devices in the design (@User and several others): I just pushed an update to open_pdks that makes changes to the PNP devices that are saved in the sky130_fd_pr library. I changed the name of the layout to add

_rf_

to the name of the layout view; e.g.,

sky130_fd_pr__rf_pnp_W3p40L3p40.gds

. This became necessary because when I updated the

.mag

view to allow it to write the correct name of the subcircuit wrapper for the PNP model, which is named

sky130_fd_pr__pnp_W3p40L3p40

, I realized that the layout has to have a different name than the subcircuit wrapper or else the two subcircuit names will be the same and you'll end up with an output in which the subcircuit is apparently instantiating itself (oddly enough, the NPNs were all correctly done already). If you have something hacked together that works now, then probably you don't want to mess with it, but if you want to do the update I can explain how to do it with minimal disruption.