search

hiveeyes / hanimandl

HaniMandl ist ein halbautomatischer Honig-Abfüll-Roboter.
https://hanimandl.readthedocs.io/
GNU General Public License v3.0
38 stars 29 forks source link

Update library versions #17

Closed amotl closed 3 years ago

amotl commented 3 years ago

Hi Clemens and Andreas,

this uses the most recent versions of the U8g2 and ESP32Servo libraries.

Both libraries are well maintained by their respective authors and maintainers, so we should use them in favour of outdated libraries which are no longer maintained.

A little adjustment has been made to one #include statement. The updated library header file is now called ESP32Servo.h instead of ESP32_Servo.h.

This also corresponds with 8e605aa5 by @andreash-esp.

With kind regards, Andreas.

amotl commented 3 years ago

You have to also adjust the servo defaults.

Thanks. I will amend this patch accordingly.

ClemensGruber commented 3 years ago

Ich möchte im code nicht auf platformio als Bibliotheks-Quelle hinweisen, wenn die Bibliothek über die Arduino-IDE installiert werden kann. Wir haben u.a. auch die ESP32-Servo-Bibliothek gewechselt, damit Leute die einfach(er) über die Arduino-IDE einbinden können.

Wenn jemand platformio verwendet sind die Dinge in platformio.ini hinterlegt und man muss sich eh nicht kümmern, daher finde ich die Änderung der Kommentare hier kontraproduktiv.

#include <U8g2lib.h> /* aus dem Bibliotheksverwalter */ vs #include <U8g2lib.h> /* https://platformio.org/lib/show/942/U8g2 */

amotl commented 3 years ago

Ich möchte im code nicht auf PlatformIO als Bibliotheks-Quelle hinweisen.

Dann nehme ich das wieder zurueck.

amotl commented 3 years ago

Hi Clemens,

ich habe den Patch ensprechend angepasst.

Viele Gruesse, Andreas.