search

veista / nilan

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

Add passive air exchanger efficiency for all devices #18

Closed gllmlbrt closed 1 year ago

gllmlbrt commented 1 year ago

I think this sensor is relevant for all machines, but feel free to restrict it only to COMFORT. Tested and works fine. image

gllmlbrt commented 1 year ago

Will do the change. "air_temp_eff_pct" is the term used by Nilan in the protocol, otherwise I agree with you.

veista commented 1 year ago

Yes, they are not easy to remember or very describing.

veista commented 1 year ago

By the way, my device does not have a passive exchanger and the efficiency still shows. Thanks!

gllmlbrt commented 1 year ago

my device does not have a passive exchanger and the efficiency still shows. Thanks!

Maybe we should then only point to the COMFORT series (13) ?

Changes done now.

veista commented 1 year ago

my device does not have a passive exchanger and the efficiency still shows. Thanks!

Maybe we should then only point to the COMFORT series (13) ?

Changes done now.

No, I mean that it works! Did not notice this feature.

AdamGrumlik commented 1 year ago

I wondered what to add the air calculation too?

DPT.AirFlow1 2201 m3/h Calculated air flow 1 DPT.AirFlow2 2202 m3/h Calculated air flow 2