Modified hardcoded PDK path with the environment variable PDK_ROOT in sky130_mapped.py

[iitkgpshubham]

• Modified hardcoded PDK path with the environment variable PDK_ROOT in sky130_mapped.py
• PDK_ROOT also exist in docker installation, see Line 25 docker/conda/Dockerfile ENV PDK_ROOT=/usr/bin/miniconda3/share/pdk/
• PDK_ROOT also created in Line 305 dependencies.sh export PDK_ROOT=/home/$(logname)/miniconda3/share/pdk/
• PDK_ROOT variable is also used by other open source tools

[iitkgpshubham]

@msaligane @chetanyagoyal The test are failing ImportError: numpy.core.multiarray failed to import.
I can see that tests uses numpy 2.0.0 Requirement already satisfied: numpy in /usr/bin/miniconda3/lib/python3.10/site-packages (from -r requirements.txt (line 3)) (2.0.0)

I use a version 1.26.4. I can see that docker also uses a same version. Please see if this can be used.

[chetanyagoyal]

@msaligane @chetanyagoyal The test are failing ImportError: numpy.core.multiarray failed to import. I can see that tests uses numpy 2.0.0 Requirement already satisfied: numpy in /usr/bin/miniconda3/lib/python3.10/site-packages (from -r requirements.txt (line 3)) (2.0.0)

I use a version 1.26.4. I can see that docker also uses a same version. Please see if this can be used.

We are handling this in a separate PR

[chetanyagoyal]

This has a lot of potential to fail. Please enclose all logic in a try and except block instead, where if the pdk_root can't be found, it defaults to the two possible locations (/usr/bin/... or /home/$(logname)/...)

[iitkgpshubham]

Can you check if it makes sense now. However it is expected that PDK_ROOT variable gets created during installation...as docker installation does it.

[iitkgpshubham]

@chetanyagoyal please see if the new changes makes sense. However it is expected that PDK_ROOT variable gets created during installation...as docker installation does it.

[chetanyagoyal]

Let's see if the workflows pass