Closed SenH closed 1 year ago
Bedankt voor je melding. In v5.10 is de onderliggende parser bijgewerkt, juist om later ondersteuning te bieden voor dat veld.
Echter ontbreekt er nog zo'n telegram in de tests voor DSMR-reader, dus achteraf had ik die beter eerst kunnen toevoegen.
Ik zal kijken of de fix makkelijk is. Hoe dan ook kun je overwegen om te downgraden naar v5.9 tot die tijd.
Oke de fix is relatief simpel voor DSMR-reader, namelijk dat veld hard in de parser negeren, gezien DSMR-reader er toch (nog) niets mee doet.
Ik zal kijken of ik er een v5.10.3 fix voor kan maken.
Ik wacht nog even de fix af dan voor ik downgrade. Buiten de error logs draait het wel verder.
Voor zover ik de error begrijp zou de MaxDemandParser
class moeten aangeroepen worden om dit veld te parsen in plaats van de MBusParser
.
Bedankt voor je pointer, ik heb hem aan #1764 toegevoegd voor later. Voor nu is die regel hard uit de kopie van de parser in DSMR-reader gehaald.
Als alle tests slagen dan zal ik hem mergen en releasen: https://github.com/dsmrreader/dsmr-reader/actions/runs/4068640849
Ik heb je telegram met een random serienummer toegevoegd als test, zodat dit een volgorde keer niet onopgemerkt gebeurt: https://github.com/dsmrreader/dsmr-reader/commit/d88cbe8d4be551d17a7bb61d5f6ddb27ad640b56#diff-583616bc5af1bf7227f1124872ce725a3d5703dc452721874a85a8f50a6e874bR25
Mijn slimme meter is nog maar een paar dagen naar v1.7 (waar die velden met kwartier piek waardes inzitten) ge-upgrade. Het zou kunnen dat de error zich al vroeger manifesteerde maar ik bekijk de logs van DSMR-Reader niet elke dag want het draait verder als een zonnetje. Veel dank alvast!
De vorige versie van DSMR-reader (of de gebruikte parser) kende dat veld niet, dus ik denk dat het veld eerder genegeerd werd.
Het is uitgebracht: https://github.com/dsmrreader/dsmr-reader/releases/tag/v5.10.3
Wellicht dat @xirixiz morgenavond de bijbehorende Docker-release voor je wilt taggen
Released!
Thanks!
Description
Volgende error in Docker log om de de 5 sec.
Lijkt iets niet goed te gaan met het parsen van 1.6.0 register (België - Sibelga)? Zie pag. 9 e-MUCS_P1_Ed_1_7_1.pdf
DSMR-reader version
v5.10.2
DSMR-reader platform
Docker (e.g. Xirixiz)
Debug info dump