docs-content icon indicating copy to clipboard operation
docs-content copied to clipboard

[MKC-667]benjamindannegard/nano33 TF tutorial update

Open BenjaminDannegard opened this issue 2 years ago • 7 comments

What This PR Changes

  • Updated Tensorflow tutorial since the library was removed from our library manager

Contribution Guidelines

BenjaminDannegard avatar Sep 23 '22 13:09 BenjaminDannegard

:white_check_mark: docs-content deploy preview ready

gatsby-cloud[bot] avatar Sep 23 '22 14:09 gatsby-cloud[bot]

@BenjaminDannegard can you take a look at the comments from Per and test it out and adjust the content?

karlsoderby avatar Oct 03 '22 10:10 karlsoderby

The IMU Classifier sketch at the end of the tutorial fails to compile when using the updated library version:

C:\Users\per\Documents\Arduino\IMUClassifier\IMUClassifier.ino:23:10: fatal error: tensorflow/lite/version.h: No such file or directory
 #include <tensorflow/lite/version.h>
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

exit status 1

Compilation error: tensorflow/lite/version.h: No such file or directory

As you can see, the library does not contain this file:

https://github.com/tensorflow/tflite-micro-arduino-examples/tree/4056e5f13aafb4dd3eda46cec273133a1d5e9593/src/tensorflow/lite

I also found there is a problem with the "Setup Python Environment" step of the Jupyter notebook:

ERROR: Could not find a version that satisfies the requirement tensorflow==2.0.0-rc1 (from versions: 1.13.1, 1.13.2, 1.14.0, 1.15.0, 1.15.2, 1.15.3, 1.15.4, 1.15.5, 2.0.0, 2.0.1, 2.0.2, 2.0.3, 2.0.4, 2.1.0, 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2.0, 2.2.1, 2.2.2, 2.2.3, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.4.1, 2.4.2, 2.4.3, 2.4.4, 2.5.0, 2.5.1, 2.5.2, 2.5.3, 2.6.0rc0, 2.6.0rc1, 2.6.0rc2, 2.6.0, 2.6.0+zzzcolab20220506153740, 2.6.1, 2.6.2, 2.6.3, 2.6.4, 2.6.4+zzzcolab20220516125453, 2.6.5, 2.6.5+zzzcolab20220523104206, 2.7.0rc0, 2.7.0rc1, 2.7.0, 2.7.0+zzzcolab20220506150900, 2.7.1, 2.7.2, 2.7.2+zzzcolab20220516114640, 2.7.3, 2.7.3+zzzcolab20220523111007, 2.7.4, 2.8.0rc0, 2.8.0rc1, 2.8.0, 2.8.0+zzzcolab20220506162203, 2.8.1, 2.8.1+zzzcolab20220516111314, 2.8.1+zzzcolab20220518083849, 2.8.2, 2.8.2+zzzcolab20220523105045, 2.8.2+zzzcolab20220527125636, 2.8.2+zzzcolab20220629235552, 2.8.2+zzzcolab20220714152931, 2.8.2+zzzcolab20220714162028, 2.8.2+zzzcolab20220719082949, 2.8.2+zzzcolab20220822160911, 2.8.3, 2.9.0rc0, 2.9.0rc1, 2.9.0rc2, 2.9.0, 2.9.1, 2.9.2, 2.10.0rc0, 2.10.0rc1, 2.10.0rc2, 2.10.0rc3, 2.10.0)
ERROR: No matching distribution found for tensorflow==2.0.0-rc1

@per1234 could we get in contact with the Tensorflow people and ask them if they can take a look at the library issue and fix it on their end? Do you know anyone over there we could talk to about this?

BenjaminDannegard avatar Oct 03 '22 12:10 BenjaminDannegard

Hi @BenjaminDannegard. I don't know anyone involved with the TensorFlow project. I only received the library removal request from them as part of my duties maintaining the Arduino Library Manager index as a whole.

per1234 avatar Oct 03 '22 13:10 per1234

@BenjaminDannegard can you make a copy of the now incorrect example, and then remove it from the article for now? Like we discussed earlier.

karlsoderby avatar Oct 06 '22 13:10 karlsoderby

@BenjaminDannegard I manually added in version.h which I got from here: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/core/public/version.h

and my project compiled. if you're looking for some quick and dirty fix this may work for you.

UniversalGeneve1 avatar Nov 26 '22 21:11 UniversalGeneve1

@BenjaminDannegard @karlsoderby what is the status of this PR?

jhansson-ard avatar Oct 25 '23 09:10 jhansson-ard