hi @cammonitor !
We don’t claim to support arduino-esp32 core v3.x at the moment, as you can see from this comment our Arduino sketches for ESP32 are tested with 2.0.4.
// These sketches are tested with 2.0.4 ESP32 Arduino Core
// https://github.com/espressif/arduino-esp32/releases/tag/2.0.4
I’ll create an issue internally, meanwhile feel free to use the solution you’ve found!
Hi @cammonitor thanks for your solution. I’ve been having some issues with the touch sensor on an ESP32-S3 and I’m hoping updating to the latest Arduino library may help it. 3.x is only about 5 months old but it would be nice to get it officially supported by Edge Impulse. Making the changes you suggested appears to be sufficient to completely use edge-impulse-sdk.
As an alternative that more closely aligns with the rest of the code in those files, a better solution may be to designate every value, rather than removing the designations.
As these changes don’t seem to be directly related to Arduino ESP32 3.x support, rather they’re just good practice, I’d be happy to file a PR for these changes. Is the edge-impulse-sdk source repository public? There are aggregated versions of it on the Edge Impulse GitHub account but I can’t find the canonical respository.
We don’t accept PRs to edge-impulse-sdk (it is public though and repository is called inferencing-sdk) as we mostly have top-down development workflow.
I will make these changes next time ESP NN is updated in SDK (this code is from ESP NN optimized kernels rather than standard tflite micro kernels), which should be quite soon!