Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix für Wochendaten/ThingSpeak #15

Merged
merged 2 commits into from
Mar 6, 2025
Merged

Conversation

meks007
Copy link
Contributor

@meks007 meks007 commented Mar 5, 2025

Fixes für #13 und #14

@meks007 meks007 mentioned this pull request Mar 5, 2025
@meks007
Copy link
Contributor Author

meks007 commented Mar 5, 2025

Ich hab mal versucht die verwendeten Bibliotheken zu ändern und eine Firmware erstellt:

https://github.com/meks007/amis_smartmeter_reader/releases/tag/PR%2315

Reader funktioniert bei mir mit dieser Funktion aber da ich die Funktionen um die es in den Issues geht selbst nicht verwende, kann ich nicht beurteilen, ob das geholfen hat.

@mgerhard74 Kannst du bei dir die 1.3.1 neu kompilieren? Wenn ja, würde es helfen, wenn du mir sagst, welche Libraries PIO für 1.3.1 verwendet. Da die 1.3.1 scheinbar funktioniert, würde ich gerne die exakt gleichen Libs dann auch für die 1.4 verwenden. Da die Sources ansonsten ident sind, gehe ich davon aus, dass das eure Probleme behebt.

@meks007
Copy link
Contributor Author

meks007 commented Mar 5, 2025

Hab bei mir jetzt die 1.3.1 von dir geflasht und da tauchen in "Energie wöchentlich" die Daten für Heute, Gestern, Montag, Sonntag, ... sofort auf. Mit meiner 1.4.1 aus diesem PR hier ist nur Heute befüllt.

Wundert mich aber ehrlicherweise auch nicht, weil die 1.3.1 von deiner Homepage hat 641kb. Die von mir kompilierte hat 452kb. Also da ist definitiv wo der Hund drinnen. Ich muss wissen, welche Bibliotheken für 1.3.1 verwendet wurden, sonst such ich mich zu Tode.

@mgerhard74
Copy link
Owner

Hilft dir die Ausgabe von "platformio run" wenn ich 1.3.1 erstelle?

@meks007
Copy link
Contributor Author

meks007 commented Mar 5, 2025

platformio pkg list wäre besser da stehen die maintainer dabei

aktuelle ziele ich auf folgende ab, ist aber vermutlich komplett verkehrt:

Libraries
├── ArduinoJson @ 5.13.4 (required: bblanchon/ArduinoJson @ 5.13.4)
├── AsyncMqttClient @ 0.9.0 (required: marvinroger/AsyncMqttClient @ 0.9.0)
│ ├── AsyncTCP @ 1.1.1 (required: AsyncTCP @ >=1.1.1)
│ └── ESPAsyncTCP @ 1.2.2 (required: ESPAsyncTCP @ >=1.2.2)
├── AsyncPing(esp8266) @ 1.1.0 (required: akajes/AsyncPing(esp8266))
├── ESP Async WebServer @ 1.2.3 (required: zeed/ESP Async WebServer @ 1.2.3)

@mgerhard74
Copy link
Owner

Core v6.1.16 is used (previous was 6.1.17)
Please remove multiple PIO Cores from a system:
https://docs.platformio.org/en/latest/core/installation/troubleshooting.html


Resolving esp12e dependencies...
Platform espressif8266 @ 4.1.0 (required: espressif8266)
├── framework-arduinoespressif8266 @ 3.30101.0 (required: platformio/framework-arduinoespressif8266 @ ~3.30101.0)
├── framework-esp8266-nonos-sdk @ 2.1.0 (required: platformio/framework-esp8266-nonos-sdk @ >=2.1.0)
├── tool-esptool @ 1.413.0 (required: platformio/tool-esptool @ <2)
├── tool-esptoolpy @ 1.30000.201119 (required: platformio/tool-esptoolpy @ ~1.30000.0)
├── tool-mklittlefs @ 1.203.210628 (required: platformio/tool-mklittlefs @ ~1.203.0)
├── tool-mkspiffs @ 1.200.0 (required: platformio/tool-mkspiffs @ ~1.200.0)
└── toolchain-xtensa @ 2.100300.210717 (required: platformio/toolchain-xtensa @ ~2.100300.0)

Libraries
├── ArduinoJson @ 5.13.4 (required: ArduinoJson @ 5.13.4)
├── AsyncMqttClient @ 0.9.0 (required: AsyncMqttClient)
│ ├── AsyncTCP @ 1.1.1 (required: AsyncTCP @ >=1.1.1)
│ └── ESPAsyncTCP @ 1.2.2 (required: ESPAsyncTCP @ >=1.2.2)
├── ESP Async WebServer @ 1.2.3 (required: ESP Async WebServer)

@meks007
Copy link
Contributor Author

meks007 commented Mar 5, 2025

Mist, steht nicht dabei. Kannst du in PIO in die Bibliotheken reingehen und dort die installierten Bibliotheken für das Projekt abtippen?

grafik

@mgerhard74
Copy link
Owner

ArduinoJson by Benoit Blanchon 5.13.4
AsyncMqttClient by Marvin ROGER 0.9.0
AsyncTCP by Hristo Gochkov 1.1.1
ESP Async WebServer by Hristo Gochkov 1.2.3
ESPAsyncTCP by Hristo Gochkov 1.2.2

@meks007
Copy link
Contributor Author

meks007 commented Mar 5, 2025

Eigentlich ident. Kannst du die Firmware compilen und mir sagen, wie groß das bin-File ist? Eventuell gleich hochladen, dann kann ich's testen was passiert.

Mach bitte auch einen Screenshot vom libdeps Ordner, sollte ähnlich aussehen wie bei mir (ohne der Ping-Bibliothek):

grafik

@mgerhard74
Copy link
Owner

Hier das Bild und meine eben nochmals erstellte firmware.bin (1.3.1).
grafik
firmware.zip

@meks007
Copy link
Contributor Author

meks007 commented Mar 5, 2025

Danke, ich denke jetzt hab ich alles. Ich melde mich wenn ich mehr weiß.

consolidate ping restart ticker into main loop
@meks007
Copy link
Contributor Author

meks007 commented Mar 5, 2025

Apologies, Fehler gefunden. Lag tatsächlich am neuen Ping Restart Check von der 1.4. Mir ist noch nicht ganz klar, weshalb, aber der zweite Ticker den ich für die asynchronen Pings gestartet habe, hat mit dem Main Loop konkurriert. Dadurch wurde dann die Historie und auch ThingSpeak in Mitleidenschaft gezogen.

Bitte mit der PR#15-0.2 noch einmal testen:
https://github.com/meks007/amis_smartmeter_reader/releases/tag/PR%2315

Bei mir siehts so aus:
grafik

@meks007
Copy link
Contributor Author

meks007 commented Mar 5, 2025

PS: Ich vermute stark (zumindest macht es den Anschein), dass die Statistiken seit 1.4 auch nicht mehr korrekt geschrieben worden sind. Das heißt die Daten der ~letzten Woche sind mit Sicherheit verzerrt.

@mgerhard74 mgerhard74 merged commit 949b172 into mgerhard74:main Mar 6, 2025
@mgerhard74
Copy link
Owner

Hallo @meks007 , @k3067e3,
Ich habe eine v1.4.2 erstellt. Update von 1.4.1: nur firmware.bin, von 1.3.1: firmware.bin, cust.js und index.html.
Die neue Firmware.bin hat "nur" 463kB.
Ich bemerke leider noch immer ein komisches Verhalten der Website. Bei mir funktionieren Wochenansicht und Thingspeak Upload. Allerdings wird manchmal nicht die komplette Seite in LOG und STATUS angezeigt. Verhält sich (manchmal) etwas komisch. Ich habe Wifi Sleep Mode aus und Neustart bei Ping Fehler aus.
lg Gerhard

@meks007
Copy link
Contributor Author

meks007 commented Mar 6, 2025

Habs bei mir jetzt ein wenig probiert, funktioniert aber eigentlich normal. Wäre gut wenn du das ein wenig eingrenzen könntest.

@mgerhard74
Copy link
Owner

Wie User @k3067e3 in Issue #14 schreibt, habe das gleiche Verhalten:
Ich habe jetzt auch den Ping Reboot aktiviert und neu gestartet. Damit ist in der Wochenansicht nur mehr heute sichtbar und der Thingspeak Upload findet nicht mehr statt.

@meks007 meks007 deleted the fix-wd+ts branch March 7, 2025 13:59
@meks007
Copy link
Contributor Author

meks007 commented Mar 7, 2025

Ja, gerade getestet - die 1.4.2 die du veröffentlich hast, hat den Fehler nach wie vor. Die Firmware von meinem PR#15 hat den Fehler allerdings nicht mehr. Ich würde vorschlagen, dass du die Firmware neu compilest und neu hochlädst.

@meks007
Copy link
Contributor Author

meks007 commented Mar 7, 2025

Als Gegenprobe kannst du gerne testweise die von mir gerade kompilierte 1.4.2 herunterladen und testen:
https://github.com/meks007/amis_smartmeter_reader/releases/tag/PR%2315

@mgerhard74
Copy link
Owner

mgerhard74 commented Mar 7, 2025

Ich hatte den Quellcode deiner PR#15 für meine 1.4.2 verwendet - hmmm....
Ich machs nochmals, da sind Unterschiede....

@mgerhard74
Copy link
Owner

Danke @meks007 ,
habe die 1.4.3 erstellt, sieht sehr gut aus!
lg Gerhard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants