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
Add support for multiple mbus devices in dsmr #84097
Conversation
Hey there @RobBie1221, @frenck, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
32201cb
to
6dbd691
Compare
6dbd691
to
518efd1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make sure CI passes.
d99e225
to
7613c7f
Compare
7613c7f
to
35b67e2
Compare
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as off-topic.
This comment was marked as off-topic.
72c8244
to
a8f4fe7
Compare
@frenck : CI passes and branch updated to latest dev. Its running fine here on my own HA. I hope this can get merged before the 2023.2.0 release as this is a hot-topic in Belgium and a lot of people will find this very useful! |
b7b8591
to
95ed95a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments.
May be we should also migrate the DSMR 5 gas meter to an MBUS entity in a follow up PR
Further I think we should no longer add the devices during the config flow, but add them dynamically instead and notify the user that any device will be detected automatically.
A dsmr meter can have 4 mbus devices. Support them all and also add support for a water meter on the mbus device.
Co-authored-by: Jan Bouwhuis <jbouwh@users.noreply.github.com>
13f4cb1
to
51772bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice!
Thnx @dupondje 👍
Only thing that is needed is a DOC's PR to mention the water meter capabilities etc
CI fails on unrelated code |
@frenck Do you approve as code owner? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please address the comment in a new PR. Thanks!
Breaking change
For users using the 5B version:
The previous gas sensor (Gas consumption) will change to Gas consumption mbusX.
Also multiple gas sensors will be possible.
Proposed change
Type of change
Additional information
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.To help with the load of incoming pull requests: