-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[luxtronikheatpump] Add power consumption channel #18254
Conversation
Signed-off-by: Clemens Eberwein <[email protected]>
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.
My heatpump doesn't have the latest version, so I'm not able to test this new channels.
The changes however look good overall, left two comments for improvements/changes.
bundles/org.openhab.binding.luxtronikheatpump/src/main/resources/OH-INF/thing/channels.xml
Outdated
Show resolved
Hide resolved
...hab.binding.luxtronikheatpump/src/main/resources/OH-INF/i18n/luxtronikheatpump_de.properties
Outdated
Show resolved
Hide resolved
…es/OH-INF/thing/channels.xml Co-authored-by: Stefan Giehl <[email protected]> Signed-off-by: negesti <[email protected]>
Signed-off-by: Clemens Eberwein <[email protected]>
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.
One minor comment, Otherwise LGTM
...penhab.binding.luxtronikheatpump/src/main/resources/OH-INF/i18n/luxtronikheatpump.properties
Outdated
Show resolved
Hide resolved
…n i18n, remove duplicate channel from README.md Signed-off-by: Clemens Eberwein <[email protected]>
… existing Hot Gas Temperature channels Signed-off-by: Clemens Eberwein <[email protected]>
Sry for the poor quality of the PR and wasting your time :( I re-generated the properties using the i18n plugin as @jlaur suggested and updated label for the label for |
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.
Thanks, LGTM
Description
With Firmware V3.90.x (not sure about the exact version) addional channels are available (261-268)
Especially channel 268 "Power Consumption" is now avaible and gives details about the actual (electric) power consumption of the heatpump.
Testing
Tested the new channels with a local docker installation for the last ~ 12 hours without any errors/exceptions