-
Notifications
You must be signed in to change notification settings - Fork 17
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
Return of separate upper/lower hysteresis options? #82
Comments
I too would also welcome this feature back. I used the upper and lower to control water temperature of my gas heated hot tub, but also for wireless TRV's like you. I have noticed the thermal overshoot now without the ability to vary the hysteresis. Please consider bringing it back. |
I really enjoyed using this heater control, unfortunately this year I upgraded the node red core and had to update the heater-controller to 3.0.2 as well. I miss the hysteresis lower and upper limits a lot. |
Finally had a spare moment and some motivation to fork this seemingly abandoned code (original author: please forgive my assumption if it is incorrect - as stated, I love your node!). I've hacked away and have seemingly restored the ability to set upper and lower hysteresis values....... See my fork here: Instructions that work for me DO THIS AT YOUR OWN RISK
Obviously if it doesn't work for you, simply re-install the current version of your choice: If you want the older version 2 with functioning upper/lower hysteresis: If you want the original author's current (latest) version without upper/lower hysteresis: Hope this helps someone 😸 |
Firstly, excellent ui node!
This is an enhancement request, not a bug (actually a return of a previous feature from v2).
As per the title - I am currently using v2.02 with a plan to move to v3 (been testing on a node-red test instance) but one feature that I personally make use of is the ability to set separate upper and lower hysteresis values.
I use your heater controller node (one for each z-wave thermostatic radiator valve, 19 in total) to drive the call for heat from our boiler. Based on the output value of currentHeaterStatus which I store in persistent global variables (one per valve), a simple boolean logic node checks the global variables and if one or more are in state "on", the boiler is fired.
Because of thermal overshoot in the winter (the boiler alters the flow temperature based on an external "weather compensation" sensor - AKA a thermistor!) when the flow temp can be as high as 70c, I use an upper hysteresis value of between -0.2 and -1, dependent on the size of the room/radiator.....the bigger the radiator and smaller the room, the greater the thermal mass and larger the overshoot from the 70c water. I also use a lower hysteresis of between 0.5 and 1, again dependent on the room/radiator size. This for the most part has drastically reduced the thermal overshoot as the heater outputs "off" between 0.2 and 1 degree below the target temperature.
The new heater node has a single hysteresis value that is either added or subtracted to the target temperature to determine the on/off output state.
Would you consider reintroducing the separate upper/lower settings?
I appreciate you may have no motivation to do this as this is most likely a fringe case, but I thought I'd ask 😃
The text was updated successfully, but these errors were encountered: