search

efabless / openlane2

The next generation of OpenLane, rewritten from scratch with a modular architecture
https://openlane2.readthedocs.io/
Apache License 2.0
168 stars 30 forks source link

Please update `rich` and `klayout` dependencies #494

Open hpretl opened 4 days ago

hpretl commented 4 days ago

Description

Hi @donn could you please update the rich dependency to be compatible with cace from @mole99:

image

Proposal

Upgrading to rich < 14 would be ideal.

donn commented 4 days ago

Looking into it

hpretl commented 3 days ago

BTW, when you are looking at this: Bumping KLayout to <=0.92.2 (or at least 0.91.1) would also be quite helpful! :-)

kareefardi commented 3 days ago

Rich issue addressed by https://github.com/efabless/openlane2/pull/495

hpretl commented 2 days ago

Thank you, the issue regarding rich looks solved, openlane and cace can now coexist, thanks!

Unfortunately, there is still an incompatibility with kfactory (an important piece of gdsfactory) on klayout, please see here:

33.63 ERROR: Cannot install kfactory[git,ipy]==0.17.4 and openlane==2.0.10 because these package versions have conflicting dependencies.
33.63
33.63 The conflict is caused by:
33.63     openlane 2.0.10 depends on klayout<0.29.0 and >=0.28.17.post1
33.63     kfactory[git,ipy] 0.17.4 depends on klayout>=0.29.1
hpretl commented 2 days ago

@donn @kareefardi Would an update of the dependency to klayout<=0.29.1 work? Or better yet, 0.29.2 (the latest KLayout release)?

hpretl commented 2 days ago

I tested with klayout 0.29.2, and it runs fine on the smoke test.

hpretl commented 2 days ago

See https://github.com/efabless/openlane2/pull/496