tflite_micro_compiler icon indicating copy to clipboard operation
tflite_micro_compiler copied to clipboard

Update required define

Open rafzi opened this issue 5 years ago • 6 comments

From SIGMICRO list:

If you explicitly have TF_LITE_STATIC_MEMORY defined in a Make or build config, please update that build define to TF_LITE_MICRO.

rafzi avatar Nov 12 '20 15:11 rafzi

Hi Rafael, do you have a more specific link? I was unable to find it at https://gitter.im/tensorflow/sig-micro . I would like to understand which tf versions are affected how.

cpetig avatar Nov 14 '20 12:11 cpetig

I see tensorflow/lite/micro/tools/make/Makefile in r2.4 still define TF_LITE_STATIC_MEMORY and I see no usage of a TF_LITE_MICRO in r2.4 (as of today)

cpetig avatar Nov 14 '20 13:11 cpetig

I just copied from inbox to issue, but you are right that there seems to be no mention of this flag in the official repo.

https://groups.google.com/a/tensorflow.org/g/micro/c/ExZmKoRzeAs

rafzi avatar Nov 14 '20 13:11 rafzi

That sounds like r2.5 material (also given the time it got posted (after releasing 2.4-rc2)), so I believe it won't affect us for another months (till we create a r2.5 branch). By the way would you agree to remove the branch master from the repository? Besides the obvious issues with the name I think it adds more confusion than it helps. Especially since I created r2.1..r2.4 and made r2.3 the default branch some time ago.

cpetig avatar Nov 14 '20 14:11 cpetig

The master branch still serves a purpose for me, in that I'd use it to keep up with tensorflow master. However we could rename it to "develop"/"dev" to avoid confusion.

rafzi avatar Nov 16 '20 07:11 rafzi

Today I found the info that this rename got delayed: https://groups.google.com/a/tensorflow.org/g/micro/c/ExZmKoRzeAs/m/XoemFLMgAAAJ?pli=1

cpetig avatar Dec 09 '20 21:12 cpetig