-
Notifications
You must be signed in to change notification settings - Fork 25
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
electricity, water, gas > Belgian meter #32
Comments
Hi, great addition, thanks! But can you please make a Pull Request for this? And please also add a test scenario including an example of the P1 message of your meter. That way we can make sure we can validate it will stay working in future changes. Thanks! |
Hey RuudI am not so good in github. Tried to make a pull request, but could not find out how...DidierVerzonden vanaf mijn Galaxy
-------- Oorspronkelijk bericht --------Van: Ruud Verheijden ***@***.***> Datum: 15/03/2023 21:14 (GMT+01:00) Aan: ruudverheijden/node-p1-reader ***@***.***> Cc: dwindey1 ***@***.***>, Author ***@***.***> Onderwerp: Re: [ruudverheijden/node-p1-reader] electricity, water, gas > Belgian meter (Issue #32)
Hi, great addition, thanks! But can you please make a Pull Request for this? And please also add a test scenario including an example of the P1 message of your meter. That way we can make sure we can validate it will stay working in future changes. Thanks!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>
|
I seems that this is not yet in the official release? |
For example this E-meter has 4 watermeters on spread over multiple channels
|
It's been a while that I tweeked this code, so I don't realy recall any details. |
I just got my digital meter (Belgian). We also got a digital watermeter and gas meter installed.
The code needed to be tweeked a bit for getting the data for the 3 meters.
here is the working code
The text was updated successfully, but these errors were encountered: