-
-
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
Add temperature dependence for Best Save module #153
Comments
How would that work? |
You could enter a Max sequence and Min recovery for a given temperature differance between indoor and outdoor, say 20 deg C, and the value would be weighted by the actual temperature difference between in and outdoor temps.
If delta T is 40 degrees, the Max sequence would be halved and the Min recovery doubled.
Skickat från Outlook för Android<https://aka.ms/AAb9ysg>
…________________________________
Från: Otto Paulsen ***@***.***>
Skickat: lördag 28 januari 2023 16:43
Till: ottopaulsen/node-red-contrib-power-saver ***@***.***>
Kopia: rbrakhya ***@***.***>; Author ***@***.***>
Ämne: Re: [ottopaulsen/node-red-contrib-power-saver] Add temperature dependence for Best Save module (Issue #153)
How would that work?
—
Reply to this email directly, view it on GitHub<#153 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVJID4VL53GVWXXH7POVB53WUU5C7ANCNFSM6AAAAAAUJWMREY>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
This feature is too special to be a general feature on the node. I suggest you make your logic for this outside the node and use dynamic config or override to get what you want. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello!
Thank you for your great modules!
I think it would be nice if you could add a temperature dependency to the Best save module. The Max per sequence and Min recover depend on the outdoor temperature, so letting it influence the settings would optimise the savings even more.
:)
The text was updated successfully, but these errors were encountered: