OpenInterpreter / 01

The #1 open-source voice interface for desktop, mobile, and ESP32 chips.
https://01.openinterpreter.com/
GNU Affero General Public License v3.0
4.92k stars 517 forks source link

Arduino libraries clarification request #187

Open hpsaturn opened 6 months ago

hpsaturn commented 6 months ago

Summary

The current documentation is not clear regarding the right libraries to use. We have two sections, two readme files with this information, but also we have the header of the .ino file that mentioned different libraries. I tested with both and I had the same results, then, don't worries for that, but maybe we should define better these information.

On the other hand, PlatformIO is more clear with that. In the platformio.ini that I pushed, we could put the strict version of the libraries from the PlatformIO registry. For instance I have:

[env:m5echo]
extends = esp32common
lib_deps = 
  m5stack/M5Atom @ ^0.1.2
  links2004/WebSockets @ ^2.4.1
  https://github.com/arduino-libraries/ArduinoHttpClient
  https://github.com/lacamera/ESPAsyncWebServer
  fastled/FastLED @ ^3.6.0

(but, I repeat, we need more clarification for these libraries, specifically regarding ArduinoHttpClient and ESPAsyncWebServer versions)

Curent Arduino ino file header libraries info:

#include <AsyncTCP.h> //https://github.com/me-no-dev/AsyncTCP using the latest dev version from @me-no-dev
#include <DNSServer.h>
#include <ESPAsyncWebServer.h> //https://github.com/me-no-dev/ESPAsyncWebServer using the latest dev version from @me-no-dev
tyfiero commented 6 months ago

Good call out, we really should clarify what arduino libraries are necessary. There is a pull request in right now to change some of this, I'll make sure these other libraries get in there. Thanks

rbrisita commented 5 months ago

Thanks for this issue. I clicked on Verify on the Arduino IDE and it output this error:

OpenInterpreter\01\software\source\clients\esp32\src\client\client.ino:6:10: fatal error: ESPAsyncWebServer.h: No such file or directory
 #include <ESPAsyncWebServer.h> //https://github.com/me-no-dev/ESPAsyncWebServer using the latest dev version from @me-no-dev
          ^~~~~~~~~~~~~~~~~~~~~
compilation terminated.
exit status 1

Compilation error: ESPAsyncWebServer.h: No such file or directory

So, @hpsaturn, it's okay to not include it?

rbrisita commented 5 months ago

Well, I added it and the IDE output was:

In file included from \Documents\Arduino\libraries\FastLED\src/FastLED.h:75,
                 from \Documents\Arduino\libraries\M5Atom\src\utility\LED_DisPlay.h:4,
                 from \Documents\Arduino\libraries\M5Atom\src\utility\LED_DisPlay.cpp:1:
\Documents\Arduino\libraries\FastLED\src/fastspi.h:157:23: note: #pragma message: No hardware SPI pins defined.  All SPI access will default to bitbanged output
 #      pragma message "No hardware SPI pins defined.  All SPI access will default to bitbanged output"
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Sketch uses 977045 bytes (74%) of program storage space. Maximum is 1310720 bytes.
Global variables use 58024 bytes (17%) of dynamic memory, leaving 269656 bytes for local variables. Maximum is 327680 bytes.

It verified, so the upload should be good.