Xilinx / finn

Dataflow compiler for QNN inference on FPGAs
https://xilinx.github.io/finn
BSD 3-Clause "New" or "Revised" License
714 stars 228 forks source link

Quicktest fail because of deprecated api call #1191

Open kalahel opened 15 hours ago

kalahel commented 15 hours ago

Hello,

Starting from a fresh install and running ./run-docker.sh quicktest I have thoses error :

[...]
 => => writing image sha256:b01e605eccedf1632bb74cdc2e8e74fd5bfe35a84f884a11ef42c597d68a246b                                        0.0s
 => => naming to docker.io/xilinx/finn:v0.10.1-dirty.xrt_202220.2.14.354_22.04-amd64-xrt                                            0.0s

 1 warning found (use docker --debug to expand):
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 144)

What's next:
    View a summary of image vulnerabilities and recommendations → docker scout quickview 
Obtaining file:///home/mathieu/Tools/finn/deps/qonnx
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [44 lines of output]
      /usr/local/lib/python3.10/dist-packages/setuptools/__init__.py:80: _DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated.
[...]

I have a lot of similar errors (deprecated api call).

After those messages, all the tests fail because they don't find qonnx, brevitas nor finn.

I tried forcing the installation by changing the dockerfile and adding :

RUN pip install qonnx
RUN pip install brevitas
RUN pip install finn-base

Now I have this test report : 7 failed, 8 passed, 19 warnings, 88 errors.

It seems that the script can't find some find API :

[...]
E   ModuleNotFoundError: No module named 'finn.transformation.qonnx'
[...]
E   ModuleNotFoundError: No module named 'finn.builder'
[...]

Did I do something wrong in my installation or is there a deprecation in the docker image ?

Thanks in advance !

auphelia commented 10 hours ago

Hi @kalahel , Are you working with current main or dev? This error looks like an older version of FINN.

kalahel commented 9 hours ago

Hi @auphelia, thanks for the response !

I followed the getting started guide so just cloned the repo with git clone https://github.com/Xilinx/finn/, so I guess I have the current main branch.