veista / nilan

Nilan integration for Home Assistant
Apache License 2.0
46 stars 11 forks source link

T4 sensor #15

Closed AdamGrumlik closed 1 year ago

AdamGrumlik commented 1 year ago

Hi, I have a feeling if there is an error with the "Waste Air Temperature" display on the Combi 302 model. As the sensor.py file has the same name for the T4 and T6 sensors. It is precisely the data from the T4 sensor that I am missing. You can look at it? Thank you.

fd

veista commented 1 year ago

Indeed. I will have to change the naming convention on these sensors since they are not consitent between models.

veista commented 1 year ago

T6 is evaporator yes, but it is the last sensor before waste in your model too. Can you confirm that the waste temperature in you HMI is actually T4? On other models with a compressor it is T6.

veista commented 1 year ago

And I dont mean the sensor name. I mean the temperature in the front panel picture.

AdamGrumlik commented 1 year ago

The unit also has a passive heat exchanger. The latter triggers the defrost based on the T4 sensor. It also has a T6. F51ADECF-F33F-4F49-A5FD-5A0055506993 33638BF4-6771-4301-B85C-90D342670DBF 47D820CC-4169-46C1-A86E-FA049F0A4F8A

AdamGrumlik commented 1 year ago

Hi. T2: Supply air sensor is now missing. On the contrary, I see T7 and the combi 302 does not have it. Maybe just misnamed.

veista commented 1 year ago

T7 == T2 it is just the register name. I do not know how it internally works, but T7 always gives a value, T2 doesn't. This doesn't relate to the original issue and will not be fixed.