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.

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

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Uprev yosys in CI to >= 0.10

alanvgreen opened this issue · comments

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.

commented

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

commented

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.

commented

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