Updated temp_sense_genCollab.ipynb #177
Conversation
Remove PATH information that is misleading and remove the python version that confuses the google collab environment. Updated the version of dependencies
|
@proppy Do you understand why the notebook and python version are breaking? |
| "!curl -Ls https://micro.mamba.pm/api/micromamba/linux-64/latest | tar -xvj bin/micromamba\n", | ||
| "conda_prefix_path = pathlib.Path('conda-env')\n", | ||
| "CONDA_PREFIX = str(conda_prefix_path.resolve())\n", | ||
| "%env CONDA_PREFIX={CONDA_PREFIX}\n", | ||
| "!bin/micromamba create --yes --prefix $CONDA_PREFIX\n", | ||
| "!echo 'python ==3.7*' >> {CONDA_PREFIX}/conda-meta/pinned\n", |
There was a problem hiding this comment.
it'd say that it is necessary to pin the python version in the environment because conda-eda only support 3.7 for now:
hdl/conda-eda#260
There was a problem hiding this comment.
oay, so that means 3.7 is fine for now? How do we make sure this is automatically updated?
There was a problem hiding this comment.
When I executed the notebook, even if I pin python 3.7, it still used python3.8 directory for some reason. But, when I remove the pin, the notebook runs smoothly. I'll check the notebook again.
There was a problem hiding this comment.
I think the problem is not on this "echo python ==3.7". The problem is some of the PATH information at the end of the code cell. I currently testing the line that caused the problem specifically.
| "!apt install -y time build-essential\n", | ||
| "!apt install qt5-default qttools5-dev libqt5xmlpatterns5-dev qtmultimedia5-dev libqt5multimediawidgets5 libqt5svg5-dev -y\n", | ||
| "!apt install ruby ruby-dev libz-dev python3-dev -y\n", | ||
| "!wget https://www.klayout.org/downloads/Ubuntu-20/klayout_0.28.3-1_amd64.deb\n", |
There was a problem hiding this comment.
note: we have a package for klayout now too!
https://anaconda.org/LiteX-Hub/klayout/files
There was a problem hiding this comment.
@Jerry-lqx and @saicharan0112 can you update to use KLayout's conda package?
There was a problem hiding this comment.
@Jerry-lqx can you remove those lines and add klayout below yosys please?
There was a problem hiding this comment.
If I edit the raw file directly, I am afraid that it might break the notebook
There was a problem hiding this comment.
Yes, I'll make the changes
I think this is because some environment change in colab (see https://medium.com/google-colab/colab-updated-to-python-3-8-4922f9970a72) we also had a similar issue in chipsalliance/silicon-notebooks#52 that we addressed with the following fix We're also adding some testing in chipsalliance/silicon-notebooks#55 (feedback welcome!), which should make detecting those kind of issues earlier. |
|
Update:
I updated %env LD_LIBRARY_PATH={LD_LIBRARY_PATH}:{CONDA_PREFIX}/lib/python3.7 Or simply, as in my previous notebook, all the path environment is replaced by os.environ['PATH'] += ":/content/conda-env/bin" This is how LDO generator attempted. |
fix outdated package dependencies, python version issue, and misleading PATH information