pihome-shc / pihome

PiHome - Smart Heating, Ventilation and Air Conditioning (HVAC)
http://www.pihome.eu
Other
52 stars 25 forks source link

Frost Protection but no status #422

Open pihome-shc opened 3 years ago

pihome-shc commented 3 years ago

i have very strange from this system, i was under the impression if i put frost protection to 0 then system will never go to frost protection mode boiler.php output

2020-11-01 20:18:26 - Zone: Sensor Reading
2020-11-01 20:18:26 - Zone: Weather Factor     0.5
2020-11-01 20:18:26 - Zone: DeadBand           0.5
2020-11-01 20:18:26 - Zone: Cut In Temperature        -0.5
2020-11-01 20:18:26 - Zone: Cut Out Temperature       0
2020-11-01 20:18:26 - Zone: Mode       21
2020-11-01 20:18:26 - Zone ID: 47
2020-11-01 20:18:26 - Zone: Landing Controller: 103 Controller Child: 1 Zone Status: 1
2020-11-01 20:18:26 - Zone: Landing Start Cause: Frost Protection - Target C:0 Zone C:

no status for frost protection icon

image

sandreialexandru commented 3 years ago

Based on this issue, the system has no frost protection off mode. It is ilogic to get frost protection at 0° C. At that temperature, the water is already frozen.

pihome-shc commented 3 years ago

Zero temperature logic was to give user option to disable frost protection but looks like deadband and weather factor makes value -

On 2 Nov 2020, at 11:03, sandreialexandru notifications@github.com wrote:

Based on this issue, the system has no frost protection off mode. It is ilogic to get frost protection at 0° C. At that temperature, the water is already frozen.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

dvdcut commented 3 years ago

some times i run into same issue but for me workaround to this is to check last temperature reporting time.

pihome-shc commented 3 years ago

i think fix is easy, change line 551 in boiler.php from if ($zone_c < $frost_c-$zone_sp_deadband){ to if (($zone_c < $frost_c-$zone_sp_deadband) AND ($frost_c!=0)) {

dvdcut commented 3 years ago

i think this should work. the en.php already have information about 0 setting,

dvdcut commented 3 years ago

this one is moved to maxair as well, i suggest to commit one line change to old version. (1.77) https://github.com/pihome-shc/PiHomeHVAC/issues/44