Skip to content

add new property for supply and return temperature of primary circuit#20

Merged
somm15 merged 1 commit into
openviess:masterfrom
michelde:feature/heatpump_props
Nov 29, 2019
Merged

add new property for supply and return temperature of primary circuit#20
somm15 merged 1 commit into
openviess:masterfrom
michelde:feature/heatpump_props

Conversation

@michelde
Copy link
Copy Markdown

The heatpumps also measure the supply and return temperature of the primary circuit. Would like to have this properties later on also in home assistant.

@crazyfx1
Copy link
Copy Markdown
Collaborator

What's the difference between your new property and the getSupplyTemperature property of the generic PyViCareDevice when the self.service.circuit is set to 0? That should also be the primary circuit.

@michelde
Copy link
Copy Markdown
Author

Using a heatpump we must distinguish between the circuit for the heating (e.g. water temperature which is flowing through the floor .) and the circuit which is going to the ground / depth (having the soil). Here I want to read the supply and return temperature.
Doing a short test shows that the values are different:

import sys
import logging
from PyViCare.PyViCareDevice import Device
from PyViCare.PyViCareHeatPump import HeatPump # heat pump
from PyViCare.PyViCareService import ViCareService

device=Device("email@domain","secret","token.save", "0")
heat=HeatPump("email@domain","secret","token.save", "0")
print("Device Supply Temp. ", device.getSupplyTemperature())
print("HeatPump Supply Temp ", heat.getSupplyTemperaturePrimaryCircuit())

returns:

Device Supply Temp.  24.4
HeatPump Supply Temp  17.7

@somm15 somm15 merged commit 6b9511d into openviess:master Nov 29, 2019
CFenner referenced this pull request in CFenner/PyViCare Dec 26, 2023
add new property for supply and return temperature of primary circuit
Nibot1 pushed a commit to Nibot1/PyViCare that referenced this pull request Aug 17, 2024
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

4 participants