Feature Request - add Mean Radiant Temperature
Closed this issue · 7 comments
colinl commented
Would you consider adding MRT as a message input value? This would be useful for me.
colinl commented
I also see that I cannot include the airspeed in the message. That would also be useful for me.
DeanCording commented
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
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
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
Ah, yes, you are right. Sorry.
DeanCording commented
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
Great, working well for me, many thanks.