espressif / esp-tflite-micro

TensorFlow Lite Micro for Espressif Chipsets
Apache License 2.0
395 stars 85 forks source link

tflite-micro is breaking complex.h (TFMIC-23) #80

Open pschatzmann opened 7 months ago

pschatzmann commented 7 months ago

I have the feeling that this issue is better addressed here.

After all it does not make any sense to have the complex.h in src since you can be sure that the standard c complex.h is available for all ESP32

maalos commented 2 months ago

have you found an easy solution yet? i just got this problem too

vikramdattu commented 2 months ago

I am not sure, how the complex.h from esp-tflite-micro is coming into picture for other projects, but most likely it's getting included from CMakeLists.txt. If you are building using lib_builder, removing this header includes from INCLUDE_DIRS should remove this issue.

do_not_include_signal_headers.patch

@maalos @pschatzmann can you please confirm if this fixes your issue?

pschatzmann commented 2 months ago

Since a regular Arduino user has nothing to do with lib_builder, I do not see how this would help.

My understanding is that Arduino is providing tflite-micro: So signal/src is added to the include path via the CMakeList of this project. Unfortunately there is a complex.h in that directory! If you don't want to delete it, I suggest to rename it.