Skip to content

Add HeatingCircuit.get{Program,Reduced,Comfort,Normal,ActiveProgram}Temperature() - #853

Open
PhrozenByte wants to merge 1 commit into
openviess:masterfrom
PhrozenByte:bugfix/heating-modes-temps
Open

PhrozenByte wants to merge 1 commit into
openviess:masterfrom
PhrozenByte:bugfix/heating-modes-temps

Conversation

@PhrozenByte

Copy link
Copy Markdown

get{Reduced,Comfort,Normal}Temperature() are shortcut methods for accessing the basic target room temperatures, complementing the existing set{Reduced,Comfort,Normal}Temperature() methods.

getProgramTemperature() replaces the now-deprecated getDesiredTemperatureForProgram().

getActiveProgramTemperature() replaces the now-deprecated getCurrentDesiredTemperature() and additionally supports holiday mode (#851). Other complex programs are not yet implemented and may still raise either PyViCareNotSupportedFeatureError or KeyError.

All temperature getters and setters now also support the program names used by reversible heat pumps (e.g., "reducedHeating" instead of "reduced").

Fixes #851

@PhrozenByte
PhrozenByte force-pushed the bugfix/heating-modes-temps branch from e39d761 to 60c78d1 Compare October 16, 2026 16:25
@PhrozenByte

Copy link
Copy Markdown
Author

Since there don't seem to be any unit tests for PyViCareHeatingDevice yet, but there are various integration tests for known devices, please let me know whether any tests should be added and, if so, what kind. Should I add unit tests, extend the integration tests for all devices, or only for a subset of devices? If the latter, which ones?

@PhrozenByte
PhrozenByte force-pushed the bugfix/heating-modes-temps branch from 60c78d1 to cab90f8 Compare October 16, 2026 17:06
…Temperature()`

`get{Reduced,Comfort,Normal}Temperature()` are shortcut methods for accessing the basic target room temperatures, complementing the existing `set{Reduced,Comfort,Normal}Temperature()` methods.

`getProgramTemperature()` replaces the now-deprecated `getDesiredTemperatureForProgram()`.

`getActiveProgramTemperature()` replaces the now-deprecated `getCurrentDesiredTemperature()` and additionally supports holiday mode (openviess#851). Other complex programs are not yet implemented and may still raise either `PyViCareNotSupportedFeatureError` or `KeyError`.

All temperature getters and setters now also support the program names used by reversible heat pumps (e.g., `"reducedHeating"` instead of `"reduced"`).

Fixes openviess#851
@PhrozenByte
PhrozenByte force-pushed the bugfix/heating-modes-temps branch from cab90f8 to 6844932 Compare October 16, 2026 17:07
@PhrozenByte PhrozenByte changed the title Add PyViCareHeatingDevice.get{Program,Reduced,Comfort,Normal,ActiveProgram}Temperature() Add HeatingCircuit.get{Program,Reduced,Comfort,Normal,ActiveProgram}Temperature() Oct 16, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vitocal 250A: Holiday mode breaks PyViCareHeatingDevice.getCurrentDesiredTemperature()

1 participant