-
Notifications
You must be signed in to change notification settings - Fork 95
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
Piekvermogen uit telegram halen #1764
Comments
Opzich heel mooi dat ze het aan het telegram toegevoegd hebben. Sterker nog, dat is uiteindelijk wel het makkelijkste. Alleen dit heeft dusdanige gevolgen dat ik even goed moet nadenken wat de eindsituatie is. Het hangt er ook erg vanaf of dit beschikbaar komt voor alle huishoudens die te maken hebben met de piekvermogens van de VREG. Sterker nog, het feit dat dezelfde gegevens te zien zijn als waar de VREG mee rekent, is nog een extra argument om mijn eigen spiegeling er uiteindelijk weer uit te halen. Ik voel er namelijk niets voor om twee mechanismes te ondersteunen. Het gaat of het ene worden of het andere. En DSMR-reader is eenmaal een DSMR reader, dus alles uit de meter heeft de voorkeur. |
Overigens heeft dit ook gevolgen voor de parsing zelf. Want |
En het gaat ook nog gevolgen hebben voor #1635, qua openstaande punten. Want eigenlijk is alle tijd die ik besteed aan deze TODO's daar zonde:
Want als het uit het telegram gaat komen, dan ga ik er sowieso een andere opslag intern voor gebruiken. Plus dat ik nog moet nadenken over welke waarden ik waar opsla (of alleen ontsluit). |
Voor de dsmr integratie gaat dit inderdaad nodig zijn. In de dsmr_reader integratie (die via MQTT de berichten van DSMR Reader ontvangt) wordt dit niet gebruikt en zal ik gewoon de nieuwe topics kunnen toevoegen. Daar verwacht ik weinig tot geen problemen mee. |
Eigenlijk is het wat zonde dat er blijkbaar niemand (ik inclusief) aan gedacht had om eens te gaan horen of er sprake was om deze info via P1 te verdelen. Dat had veel moeite en discussie kunnen besparen. Maar dat is achteraf uiteraard altijd makkelijk gezegd. |
Volledig akkoord.. ik had daar achter moeten vragen toen ik de vreg om verduidelijking vroeg.. |
Ik denk dat het verder ook niet zo 1-2-3 te verwachten was dat zoiets ingebouwd zou worden door de beheerder. Verder denk ik, praktisch gezien, dat de huidige implementatie in DSMR-reader voor de komende tijd afdoende is. Helemaal gezien de deadline van 1 januari is het mooi dat er in ieder geval "iets" te zien is voor huishoudens die voor het piekvermogen gaan betalen. Dan kan ik kijken of ik de komende maanden de nieuwe telegram data kan ontsluiten. |
In DSMR-parser zit nu ondersteuning in die ik tzt kan spiegelen. |
Zou het mogelijk zijn om in een volgende release al enkel de parser toe te voegen? |
Bedankt voor je aanvulling. Ik zal het voor de release na de aanstaande zetten. Het is namelijk wel wat meer dan alleen de parser toevoegen, gezien het ook de API, MQTT en opslag raakt. |
De parser in DSMR-reader is al wel bijgewerkt, maar de update maakt iets anders stuk. Bij latere implementaties hier naar kijken: #1799 (comment) |
Ook gemeld via #1807 (comment) |
Zeer benieuwd of dit ooit in een eerdere tussenrelease komt. Echter las ik dat dit voor september zou zijn? Dit zou bij mij ook via Home Assistant kunnen, maar heb graag alles geclubbed binnen DSMR. |
@Milithor en september is ook nog maar een planning. Oorspronkelijk wilde ik hier in het voorjaar mee bezig, maar sinds december spelen er wat kleine data-bugs in DSMR-reader waardoor de afgeleide gegevens aan het begin en eind van de dag soms afwijken. Dat fixen heeft de hoogste prio. Het huidige #1764 issue zal daarom meegaan met de volgende major release (6.x) die incompatible is met 5.x. Ik gooi dan tevens de huidige (handmatige) implementatie van het piekvermogen er dan ook uit om die te vervangen door de info uit de telegrammen. |
Kunnen wij iets doen om dit mogelijk te maken? |
@wardbeyens normaal gesproken pak ik vooral in de wintermaanden hier veel meer tijd voor, maar ik ben helaas erg druk, dus ik kan geen indicatie geven. Ook omdat een andere bug-issue nog voorrang heeft qua volgorde. |
Gemaakt vanuit #1635
@Tommatheussen:
@Glodenox :
The text was updated successfully, but these errors were encountered: