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

Belgium Fluvius Extention for Smart Metering #29

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

rvdbreemen
Copy link

This adds the belgium DSMR fields specification as a seperate namespace.
Includes the belgium specification documents.

This adds the belgium DSMR fields specification as a seperate namespace.

Includes the belgium specification documents.
@rvdbreemen
Copy link
Author

Hope this is what you would do with a name space for belgium.

To switch to beligium, just uncomment the #define
So I made it work, with a pre-compiler switch. Just because I know this works.
@rvdbreemen
Copy link
Author

rvdbreemen commented Feb 9, 2020

The dsmr fields adaption have been tested with a 3 phase Belgium meter now and a 1 phase with a gas meter. Both work according to specification.

Could you consider a merge please? Or else share with me what I should improve to have a merge become successful.

Thanks

@matthijskooijman
Copy link
Owner

I have not had time to look at this yet. I hope to have a bit more time available in a few weeks, and reviewing this (and some other issues) is on my list for then. Please hold on for a bit more :-)

@rvdbreemen
Copy link
Author

I'll hold. In the mean time the testing continues on the I1 interface, the name of the Belgium version of P1, with your great library.

@rvdbreemen
Copy link
Author

@matthijskooijman added all the telegrams I had available for you in voorbeelden.txt

@Sieberkev
Copy link

Thanks a lot @rvdbreemen for including the e-MUCS document, helped me a lot with finding the logic to several mysterious values :D

@michaelarnauts
Copy link

What's the status of this PR? I'm currently reading out my P1 with a fork of this repo (https://github.com/glmnet/arduino-dsmr/) that is used by ESPHOME), and that seems to work fine for Belgium meters, but recently, they added 1.6.0 to read out the kwartierpieken, and I would like to extract this also.

@matthijskooijman
Copy link
Owner

I still hope to properly review this PR and improve the library and its documentation in general, but realistically I'll need a fair block of time for this and there's just too much other things competing for my attention... So I'm still here, but I can't quite make any promises.

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.

4 participants