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

commonSupply is present in almost all units, but only exposed for GazBoiler #477

Open
jscheidtmann opened this issue Jan 1, 2025 · 3 comments

Comments

@jscheidtmann
Copy link

jscheidtmann commented Jan 1, 2025

Grepping the dumps in tests/response for commonSupply as exposed by GazBoiler.getBoilerCommonSupplyTemperature, this property seems to be present in almost all units.

At least for the Vitocal250, I can confirm, that it is present in the dump and also in my device. Furthermore the number seems to be shown in the German version of the App as "Vorlauftemperatur Wärmeerzeuger", right next to getReturnTemperature() ("Rücklauftemperatur Wärmeerzeuger").

Workaround to get this number - retrieve the service from any PyViCare object and query it directly

commonSupply = heatpump.service.getProperty("heating.boiler.sensors.temperature.commonSupply")["properties"]["value"]["value"]
@CFenner
Copy link
Member

CFenner commented Jan 1, 2025

Hey, having a look at the test responses, it seems that this is provided by the Vitodens devices (gas) but also by some Vitocal and one Vitovalor device. Same goes for "heating.boiler.sensors.temperature.main".
Maybe we can move this to the general HeatingDevice class.

@CFenner
Copy link
Member

CFenner commented Jan 1, 2025

What is meant by Wärmeerzeuger, is this the common supply for the heating circuits only? Why is it prefixed with boiler if it also applies to heat pumps? It's not the sensor of the bufferCylinder, no?

@jscheidtmann
Copy link
Author

jscheidtmann commented Jan 1, 2025

Wärmeerzeuger = a thing that produces heat.

For the Vitocal250A it‘s definitely not the bufferCylinder, but one of the temperatures in the pump cycle that feeds into the bufferCylinder from the heat pump. The other temp being getReturnTemperature().

I can only guess: Probably they had this id already used in gas boilers and thought it would be a good idea to re-use it. ¯\ _(ツ) _/¯

I will have a look at the other id tomorrow.

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

No branches or pull requests

2 participants