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

Plugwise incorrectly shows all zone climates hvac_action heating #103386

Closed
vincentwolsink opened this issue Nov 4, 2023 · 8 comments · Fixed by #105330
Closed

Plugwise incorrectly shows all zone climates hvac_action heating #103386

vincentwolsink opened this issue Nov 4, 2023 · 8 comments · Fixed by #105330
Assignees

Comments

@vincentwolsink
Copy link

The problem

When only one zone is heating, all Plugwise climate entities for all zones show hvac_action=heating.

What version of Home Assistant Core has the issue?

core-2023.11.0

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Plugwise

Link to integration documentation on our website

https://www.home-assistant.io/integrations/plugwise/

Diagnostics information

config_entry-plugwise-1e47758c18af3f141df39ea18083bd77.json.txt

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

home-assistant bot commented Nov 4, 2023

Hey there @CoMPaTech, @bouwew, @frenck, mind taking a look at this issue as it has been labeled with an integration (plugwise) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of plugwise can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign plugwise Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


plugwise documentation
plugwise source
(message by IssueLinks)

@bouwew
Copy link
Contributor

bouwew commented Nov 4, 2023

Hi Vincent, thanks for reporting.

You are correct, this is indeed what is happening. Plugwise has removed the control_state for each thermostat.
And we did not find anything to replace it with.

But I must have not been searching correct because a new search just shows me that that it has been moved from the thermostat-xml data to the location-xml data. Some coding to do... :)

@vincentwolsink
Copy link
Author

Thanks for your quick reply and action! I saw you already made a PR. 🚀

@bouwew
Copy link
Contributor

bouwew commented Nov 5, 2023

Yes, correct. There will be an update-release for plugwise-beta later today.
And also for the Plugwise integration in Core, but that will take some more time, there are a few updates planned to be implemented before this one.

@MartijnDisse
Copy link

Greatly appreciate the work @bouwew ! I am having the same issue.

@hugodejong
Copy link

Thanks for the work @bouwew! Any idea when this fix might be released on the HA main channel?

@bouwew
Copy link
Contributor

bouwew commented Nov 22, 2023

Solved in plugwise backend version v0.34.5

@vincentwolsink it will all depend how quickly this PR #103360 is approved and merged. This one needs to be merged before I can create a next PR that probably contains the solution for this issue.

@bouwew
Copy link
Contributor

bouwew commented Dec 10, 2023

@vincentwolsink the fix for your Issue is implemented in HA 2023.12.1

@github-actions github-actions bot locked and limited conversation to collaborators Jan 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants