Skip to content

Added heating.circuits.N.operating.modes.cooling#747

Open
bekra00 wants to merge 3 commits into
openviess:masterfrom
bekra00:add_cooling_mode_status
Open

Added heating.circuits.N.operating.modes.cooling#747
bekra00 wants to merge 3 commits into
openviess:masterfrom
bekra00:add_cooling_mode_status

Conversation

@bekra00

@bekra00 bekra00 commented Apr 20, 2026

Copy link
Copy Markdown

Added heating.circuits.N.operating.modes.cooling
to the PyViCareHeating Device and setup testing

Added heating.circuits.N.operating.modes.cooling
to the PyViCareHeating Device and setup testing
@CFenner

CFenner commented Apr 28, 2026

Copy link
Copy Markdown
Member

This can already be retrieved with getActiveMode(), not as a bool, but I think you get the idea. Do you still think we need a dedicated function for that?

@bekra00

bekra00 commented Apr 28, 2026

Copy link
Copy Markdown
Author

By getActiveMode(), I receive dhwAndHeatingCooling or heating, but not if in this moment it operates in cooling.

getActiveMode() -> heating.circuits.N.operating.modes.active
Shows current active operating mode on the device and provides command to change it.

getCoolingActive() -> heating.circuits.N.operating.modes.cooling
Shows whether the cooling operating mode is active now.

So it can be the case, that if the operation mode is set to dhwAndHeatingCooling, cooling is on or off. Therefore it does not provide the same information.

I want to use this value to open the zone valves.

@CFenner

CFenner commented Apr 29, 2026

Copy link
Copy Markdown
Member

Ok, then would it make sense to expose heating as well?

@bekra00

bekra00 commented May 29, 2026

Copy link
Copy Markdown
Author

In general yes - not for my use case but it make sense

@lackas

lackas commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

heating.circuits.N.operating.modes.cooling is device-dependent. Where cooling is a directly selectable mode it flips active: true, but where the operating mode is an umbrella (dhwAndHeatingCooling / heatingCooling) and the pump decides heating vs cooling itself, operating.modes.cooling.active stays false even while cooling runs. I verified this on a Vitocal 300-G (CU401B) doing natural cooling: the buffer dropped from 42 to 19 °C while that flag stayed false, so getCoolingActive() would report the opposite of reality there.

So this works for some setups but not all. Might be worth naming it after what it actually reads (the configured cooling mode) rather than "active", and documenting the scope. Background in #746.

@CFenner CFenner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@CFenner CFenner self-requested a review July 14, 2026 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants