search

antmicro / litex-vexriscv-tensorflow-lite-demo

TF Lite demo on LiteX/VexRiscv soft RISC-V SoC on a Digilent Arty board
https://antmicro.com/blog/2019/12/tflite-in-zephyr-on-litex-vexriscv/
Apache License 2.0
62 stars 21 forks source link

recursive clone fails #30

Open wohlbier opened 2 years ago

wohlbier commented 2 years ago 
fatal: Authentication failed for 'https://github.com/enjoy-digital/liteusb.git/'
fatal: clone of 'https://github.com/enjoy-digital/liteusb.git' into submodule path '/home/jgwohlbier/devel/litex-vexriscv-tensorflow-lite-demo/litex-buildenv/third_party/liteusb' failed
Failed to clone 'third_party/liteusb' a second time, aborting
Submodule path 'tensorflow': checked out 'fc8918faa9fa3042a709bafb8bedc5f75909b1ad'
Failed to recurse into submodule path 'litex-buildenv'
mateusz-holenko commented 2 years ago

Hi @wohlbier, this is related to the broken litex-buildenv submodule that references a non-existing liteusb repository.

As a workaround, in https://github.com/antmicro/litex-vexriscv-tensorflow-lite-demo/commit/498ef50644b620dfe8e95037321faf068b303c83 I proposed commands that should resolve the problem with cloning of submodules. Could you verify if that works for you?

mithro commented 2 years ago

Someone could send a pull request to litex-buildenv to fix the submodule....

wohlbier commented 2 years ago

I can send a PR to litex-buildenv. Thanks.

wohlbier commented 2 years ago

I sent a PR to https://github.com/antmicro/litex-buildenv. Should I have sent it to https://github.com/timvideos/litex-buildenv?

mithro commented 2 years ago

@wohlbier - The timvideos repo is probably the best idea?

wohlbier commented 2 years ago

I imagine so, but it's not clear that it is maintained. I made a PR at timvideos.

wohlbier commented 2 years ago

https://github.com/timvideos/litex-buildenv/pull/754 fixes this. Shall I do a PR to update the hash here?

mateusz-holenko commented 2 years ago

@wohlbier thanks for your involvement. It would be great if you could confirm that updating the submodule resolves the problem for you and you can successfully run the demo.

A PR fixing the state of this repo is very welcomed!

wohlbier commented 2 years ago

Ok, I'll work on that. I'm working through the mechanics of how to create the PR when the submodule (https://github.com/antmicro/litex-buildenv) does not have the changes from its base. If I demonstrate that it works when using the base (https://github.com/timvideos/litex-buildenv) is that sufficient information to show that it works? Meantime, I opened a PR for the antmicro fork at https://github.com/antmicro/litex-buildenv/pull/4, which I assume would be merged after demonstration.

wohlbier commented 2 years ago

I don't have a device yet and was just planning to do the renode simulation. In Simulating in Renode it refers to $DEMO_HOME/renode which is a path that does not exist. I'm updating the README.md and the litex-vexriscv-tflite.resc script in my fork with correct paths.

The only error I saw executing the steps was this, but it did not appear to be fatal.

Building wheels for collected packages: docopt, intervaltree, future
  Building wheel for docopt (setup.py) ... done
  Created wheel for docopt: filename=docopt-0.6.2-py2.py3-none-any.whl size=13704 sha256=66fc4705f4180766f9a47b33a6f464e0218ff8805d2be6729ba11e0cd5b57447
  Stored in directory: /home/jgwohlbier/.cache/pip/wheels/56/ea/58/ead137b087d9e326852a851351d1debf4ada529b6ac0ec4e8c
  Building wheel for intervaltree (setup.py) ... done
  Created wheel for intervaltree: filename=intervaltree-3.1.0-py2.py3-none-any.whl size=26102 sha256=46896b394d8e462c8beccfa273e9da105fcd538db09d09ec5a52589c67ae9d46
  Stored in directory: /home/jgwohlbier/.cache/pip/wheels/45/23/de/5789a92962483fd33cb06674792b9697c1b3766d7c7742830e
  Building wheel for future (setup.py) ... done
  Created wheel for future: filename=future-0.18.2-py3-none-any.whl size=491058 sha256=3c6c21a86d7170c448864fe0bcc2e26df7bade52a2822adc6ed3c8bef7c86974
  Stored in directory: /home/jgwohlbier/.cache/pip/wheels/8e/70/28/3d6ccd6e315f65f245da085482a2e1c7d14b90b30f239e2cf4
Successfully built docopt intervaltree future
ERROR: sphinx 4.5.0 has requirement docutils<0.18,>=0.14, but you'll have docutils 0.18.1 which is incompatible.
ERROR: sphinx-rtd-theme 1.0.0 has requirement docutils<0.18, but you'll have docutils 0.18.1 which is incompatible.
ERROR: sphinx-tabs 3.3.1 has requirement docutils~=0.17.0, but you'll have docutils 0.18.1 which is incompatible.
wohlbier commented 2 years ago

See https://github.com/antmicro/litex-vexriscv-tensorflow-lite-demo/pull/31