search

DeanCording / node-red-contrib-comfort

A Node Red node to calculate the thermal comfort level of an environment using ASHRAE Standard 55
Other
6 stars 4 forks source link

Feature Request - add Mean Radiant Temperature #3

Closed colinl closed 6 years ago

colinl commented 6 years ago

Would you consider adding MRT as a message input value? This would be useful for me.

colinl commented 6 years ago

I also see that I cannot include the airspeed in the message. That would also be useful for me.

DeanCording commented 6 years ago

Are you sure? I just checked and it defaults to a static number but you can change it with the drop down to a msg or context property.

I'm working on MRT at the moment.

On Wednesday, 3 January 2018 18:48:38 AEST Colin Law wrote:

I also see that I cannot include the airspeed in the message. That would also be useful for me. —You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub[1], or mute the thread[2].[3]

[1] https://github.com/DeanCording/node-red-contrib-comfort/issues/ 3#issuecomment-354961407 [2] https://github.com/notifications/unsubscribe-auth/ AAPVIl4cdAMxDGfhbf-8uGHl4xUt4BtYks5tGz7mgaJpZM4RPon1 [3] https://github.com/notifications/beacon/ AAPVIsN_IxEnMyPRL9IWgburjYw9zS83ks5tGz7mgaJpZM4RPon1.gif

colinl commented 6 years ago

Oh, no I didn't try it. I was just looking at the code which has node.airspeedField = n.airspeedField || 0; node.airspeedFieldType = n.airspeedFieldType || "num"; but I don't understand fully how this works so you may well be right. I will try it later.

DeanCording commented 6 years ago

Those are just default values if the config is missing.

You can set it to get any of the parameters to come from msg, flow, or global properties, or set them to a static value.

On Wednesday, 3 January 2018 18:56:47 AEST Colin Law wrote:

Oh, no I didn't try it. I was just looking at the code which hasnode.airspeedField = n.airspeedField || 0;node.airspeedFieldType = n.airspeedFieldType || "num";but I don't understand fully how this works so you may well be right. I will try it later. —You are receiving this because you commented.Reply to this email directly, view it on GitHub[1], or mute the thread[2].[3]

[1] https://github.com/DeanCording/node-red-contrib-comfort/issues/ 3#issuecomment-354962744 [2] https://github.com/notifications/unsubscribe-auth/AAPVIpBMF5wYITtTN-zt6nfwYvK-3e98ks5tG0DOgaJpZM4RPon1 [3] https://github.com/notifications/beacon/AAPVIiwSt-eeg7kR293TK2ecg1vr7y03ks5tG0DOgaJpZM4RPon1.gif

colinl commented 6 years ago

Ah, yes, you are right. Sorry.

DeanCording commented 6 years ago

MRT input has been added. It can be set to a property or static value, or set to operative in which case it will be the same as the air temperature.

colinl commented 6 years ago

Great, working well for me, many thanks.