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

[luxtronikheatpump] Add power consumption channel #18254

Merged
merged 5 commits into from
Feb 13, 2025
Merged

Conversation

negesti
Copy link
Contributor

@negesti negesti commented Feb 12, 2025

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

@negesti negesti requested a review from sgiehl as a code owner February 12, 2025 09:48
Copy link
Contributor

@sgiehl sgiehl left a 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.

negesti and others added 2 commits February 12, 2025 11:09
Copy link
Contributor

@lsiepel lsiepel left a 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

bundles/org.openhab.binding.luxtronikheatpump/README.md Outdated Show resolved Hide resolved
@lsiepel lsiepel added the enhancement An enhancement or new feature for an existing add-on label Feb 12, 2025
…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]>
@negesti
Copy link
Contributor Author

negesti commented Feb 13, 2025

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 temperatureHotGasMax to match the existing Hot Gas Temperature channels

Copy link
Contributor

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

@lsiepel lsiepel merged commit 5a57b9e into openhab:main Feb 13, 2025
2 checks passed
@lsiepel lsiepel added this to the 5.0 milestone Feb 13, 2025
@lsiepel lsiepel changed the title [luxtronikheatpump] added channel for power consumption [luxtronikheatpump] Add power consumption channel Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants