temp-sense generator fails with the latest tools

Question

temp-sense generator fails with the latest tools

Closed this issue a year ago · 9 comments

The temperature sensor generator is having drc and lvs errors with the latest dependencies. Attached the work folder containing the reports and the final gds file.
work.tar.gz

Answer 1 · 2023-05-08T15:16:12.000Z

https://github.com/idea-fasoc/OpenFASOC/blob/main/versions.txt
https://github.com/idea-fasoc/OpenFASOC/blob/main/conda_versions.txt

Answer 2 · 2023-05-09T07:13:07.000Z

The current official version of OpenFASOC has a syntax error in dependencies.sh. The if block that is opened in Line 26 is not closed. This causes an "unexpected end of file" error after Line 26. When i ran this on docker, the script stopped at Line 26 and didn't download anything that comes after (i.e., ngspice, klayout, xyce).

Answer 3 · 2023-05-09T14:06:52.000Z

@michahn01 I don't think these both are related ? doesn't your PR majorily overwrites this file?

Answer 4 · 2023-05-10T06:29:03.000Z

@michahn01 I don't think these both are related ? doesn't your PR majorily overwrites this file?

From what I've tested on Docker, installing OpenFASOC and running tempsensegen with my PR produces no error (clean DRC and LVS). For reference, here is the Docker file test case I used:

FROM ubuntu:20.04

RUN apt-get -y update
RUN apt-get -y upgrade
ENV DEBIAN_FRONTEND=noninteractive
RUN apt-get install -y wget git python3 vim
RUN git clone https://github.com/michahn01/OpenFASOC.git

It's necessary to run export PATH=/usr/bin/miniconda3/bin:$PATH and export PDK_ROOT=/usr/bin/miniconda3/share/pdk/after running ~/OpenFASOC/dependencies.sh inside the Docker image.

Could you share the steps of your test case when you got those DRC and LVS errors? I'll try replicating the bugs you encountered.

Answer 5 · 2023-05-10T11:56:56.000Z

Can you check if your dependencies.sh script uses the latest version of conda packages or the file named 'conda_versions.txt' file while installing them? The file 'conda_versions.txt' contains versions of dependencies that are already tested and tagged them as working.

…

On Wed, 10 May, 2023, 11:59 am michahn01, ***@***.***> wrote: @michahn01 <https://github.com/michahn01> I don't think these both are related ? doesn't your PR majorily overwrites this file? From what I've tested on Docker, installing OpenFASOC and running tempsensegen with my PR produces no error (clean DRC and LVS). For reference, here is the Docker file test case I used: `FROM ubuntu:20.04 RUN apt-get -y update RUN apt-get -y upgrade ENV DEBIAN_FRONTEND=noninteractive RUN apt-get install -y wget git python3 vim RUN git clone https://github.com/michahn01/OpenFASOC.git RUN cd OpenFASOC RUN ./dependencies.sh RUN export PATH=/usr/bin/miniconda3/bin:$PATH RUN export PDK_ROOT=/usr/bin/miniconda3/share/pdk/` — Reply to this email directly, view it on GitHub <#209 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AYCOBKMOWBIP7NCXHCFVOHDXFMYTVANCNFSM6AAAAAAX2B5TEA> . You are receiving this because you authored the thread.Message ID: ***@***.***>

Answer 6 · 2023-05-10T17:50:53.000Z

@saicharan0112

Ah, I see that I was using 'conda_versions.txt' to install the dependencies.

I retested tempsensegen in Docker with the latest versions of dependencies, and as you said the drc and lvs failed. I'll try looking more into this and I'll let you know if I find anything.

Answer 7 · 2023-05-11T06:07:17.000Z

@saicharan0112

I think the issue is with open_pdks.sky130a specifically. Using the latest version of any other package does not result in the error if open_pdks.sky130a is set to the safe version. The error happens only when using the latest version of open_pdks.sky130a.

Answer 8 · 2023-05-11T06:17:10.000Z

Yes I am already aware of that. Do you anything about those drc and lvs errors?

…

On Thu, 11 May, 2023, 11:37 am michahn01, ***@***.***> wrote: @saicharan0112 <https://github.com/saicharan0112> I think the issue is with open_pdks.sky130a specifically. Using the latest version of any other file does not result in the error if open_pdks.sky130a is set to the safe version. The error happens only when using the latest version of open_pdks.sky130a. — Reply to this email directly, view it on GitHub <#209 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AYCOBKMXRBOP64EZXQDJTTDXFR62DANCNFSM6AAAAAAX2B5TEA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

Answer 9 · 2023-05-15T16:29:53.000Z

Actual latest versions of dependencies:
Netgen: v1.5.253
Ngspice: v39.3
Klayout: v0.28.7
Yosys: v0.29+11
OpenROAD: v2.0-7559-gdf3bfc5a5
Open_pdks: v1.0.414
MAGIC: v8.3.398

Latest versions of packages available on LiteX-Hub:
Netgen: v1.5.251 (on Linux)
Ngspice: v39.3
Klayout: v0.28.6 (on Linux)
Yosys: v0.27 (on Linux)
OpenROAD: v2.0_7681_g850238c2f (on Linux)
Open_pdks: v1.0.406
MAGIC: v8.3.389