[go: nahoru, domu]

Skip to content
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

Open
bikefright opened this issue Dec 11, 2021 · 3 comments
Open

Return of separate upper/lower hysteresis options? #82

bikefright opened this issue Dec 11, 2021 · 3 comments

Comments

@bikefright
Copy link

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 😃

@chrisghz
Copy link
chrisghz commented Mar 7, 2022

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.

@jssystemcz
Copy link

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.
Please, Sergio, would it be possible to return this functionality?
Thank you very much.

@bikefright
Copy link
Author
bikefright commented Feb 3, 2024

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:
https://github.com/bikefright/node-red-contrib-heater-controller

Instructions that work for me DO THIS AT YOUR OWN RISK
(I suggest using a test instance of node-red somewhere and doing some testing with my version before putting live!)

npm install git+https://github.com/bikefright/node-red-contrib-heater-controller.git to install the node

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:
npm install node-red-contrib-heater-controller@2.0.2

If you want the original author's current (latest) version without upper/lower hysteresis:
npm install node-red-contrib-heater-controller@3.0.2

Hope this helps someone 😸

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

No branches or pull requests

3 participants