Support latest muRISCV-NN

tum-ei-eda / mlonmcu

Tool for the deployment and analysis of TinyML applications on TFLM and MicroTVM backends

Apache License 2.0

29 stars 12 forks source link

Support latest muRISCV-NN #18

Closed PhilippvK closed 2 years ago

PhilippvK commented 2 years ago

The new version of the library is going to be released soon, so we also want to use it in MLonMCU.

The integration process can be summarized as follows:

[x] Implement MLonMCU Feature
[x] Install dependencies via mlonmcu setup
[x] Integrate into MLIF
[x] Support TFLMI backend (should work out of the box with MLIF patches)
[ ] Support TFLMC backend (needs patches, generation on host architecture feasible?)
[x] Support and Test V-Extension
[x] Optional: Integration into TVM backends via CMSISNN BYOC wrappers

fpedd commented 2 years ago

Currently, some things in MURISCV-NN might be broken. Sorry about that. Will fix that soon. Should we maybe consider always using a compatible version of MURISCV-NN and TFLM by referring to certain commits? This way changes to upstream MURISCV-NN or TFLM won't break anything.

fpedd commented 2 years ago

We should also agree on an esp-idf version to use. They love to introduce breaking changes between major (and sometimes minor, arrgghh) versions (somewhat offtopic, but still relevant I guess).

So instead of

espidf:
    url: "https://github.com/espressif/esp-idf.git"
    ref: master

I would suggest

    ref: release/v4.3

Or smth similar (there already is a 4.4). But 4.3 is the version I was using last.

PhilippvK commented 2 years ago

Good ideas, thats why the environment configuration is very useful. I would definitely pin most of those repos to a version which is known to work. However I am typically working using the dev.yml.j2 template which might still point to the latest versions to make developing new features more easy.

PhilippvK commented 2 years ago

Closed because TFLMC is currently not usable with latest TFLM.