-
-
Notifications
You must be signed in to change notification settings - Fork 25
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 circadian lighting node #52
Comments
The node gives now a "altitudePercent" property which hopefully meet your needs. |
@Hypnos3 finally got a chance to test this and the output is wrong. This is what I am getting: In my example I multiple by 100 and that works but for some reason when you are doing it in your code it actually didn't need it. I don't know why this is the case (would have to dig into the code further) but for now I am just dividing by 100 until this gets fixed. Thanks for your work on this. |
Thank you, I will investigate this further. |
OK, I have found a Bug, which will fixed with the next version! |
Is your feature request related to a problem? Please describe.
Currently I am using this for circadian lighting but the flow to calculate the percent of sun in sky is very long.
Describe the solution you'd like
It would be nice if there was a node offered that gave the height of the sun in the sky in percent (0~100 with 100% being at solarnoon and 0% being completely down). Here is the flow I am using right now to get this done. If I had a node for this it would save me 5 nodes in my flow alone.
It would also be nice to be able to pass the latitude and longitude in as an input for this node. If it is not passed in use the config but if it is passed in use that instead. I want to use this with Home Assistant which already has my home zone configured. I can poll for this information and pass it into the Node so whenever I update my home area it will update in Node-RED automatically (I would also recommend adding this to other nodes as well).

Check the function blocks for more information on how my calculations are done.
Another option would be to just include this percent number in the output of the sun-position node. That way I only have to call it once instead of twice.
Describe alternatives you've considered
I have the flow that I posted above. I just think it could be easily shortened into one node. Circadian lighting node would be perfect for this library.
Additional context
I take the 0-100% and map it to a kelvin range 2500-5500. I think that part should be left to the user though and not included with this node.
The text was updated successfully, but these errors were encountered: