Uprev yosys in CI to >= 0.10

google / CFU-Playground

Want a faster ML processor? Do it yourself! -- A framework for playing with custom opcodes to accelerate TensorFlow Lite for Microcontrollers (TFLM). . . . . . Online tutorial: https://google.github.io/CFU-Playground/ For reference docs, see the link below.

http://cfu-playground.rtfd.io/

Apache License 2.0

467 stars 118 forks source link

Uprev yosys in CI to >= 0.10 #469

Closed alanvgreen closed 2 years ago

alanvgreen commented 2 years ago

The github CI seems to install yosys from Ubuntu with a pre-0.10 version number:

Get:37 http://azure.archive.ubuntu.com/ubuntu focal/universe amd64 yosys amd64 0.9-1build2 [2131 kB]

This is blocking the nMigen to Amaranth migration (#466) because Amaranth requires yosys >= 0.10.

tcal-x commented 2 years ago

Thanks for debugging @alanvgreen -- should be an easy fix.

tcal-x commented 2 years ago

Actually it's not quite what I was expecting. I didn't realize, or forgot, that our setup script would grab the yosys package via apt. It should never do that, whether in CI, or when a user is setting up.

tcal-x commented 2 years ago

I've appended commits to #466 to ./scripts/setup to do a local binary install of Yosys in if there is no system version, or if the system version is too old. This takes a few seconds.

I also considered and rejected:

Clone and build of Yosys: this took 12 minutes
Set up a Conda environment with just Yosys and then copy the files out: this took 1.5 minutes