-
Notifications
You must be signed in to change notification settings - Fork 6
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
Conversation
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. |
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. |
Hilft dir die Ausgabe von "platformio run" wenn ich 1.3.1 erstelle? |
platformio pkg list wäre besser da stehen die maintainer dabei aktuelle ziele ich auf folgende ab, ist aber vermutlich komplett verkehrt: Libraries |
Core v6.1.16 is used (previous was 6.1.17) Resolving esp12e dependencies... Libraries |
ArduinoJson by Benoit Blanchon 5.13.4 |
Hier das Bild und meine eben nochmals erstellte firmware.bin (1.3.1). |
Danke, ich denke jetzt hab ich alles. Ich melde mich wenn ich mehr weiß. |
consolidate ping restart ticker into main loop
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: |
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. |
Hallo @meks007 , @k3067e3, |
Habs bei mir jetzt ein wenig probiert, funktioniert aber eigentlich normal. Wäre gut wenn du das ein wenig eingrenzen könntest. |
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. |
Als Gegenprobe kannst du gerne testweise die von mir gerade kompilierte 1.4.2 herunterladen und testen: |
Ich hatte den Quellcode deiner PR#15 für meine 1.4.2 verwendet - hmmm.... |
Danke @meks007 , |
Fixes für #13 und #14