Piston Help - send email - THANKS!

[ldale15]

1) Give a description of the problem
The email is sending the names of all the devices without the temperature and regardless of the IF statement.

2) What is the expected behaviour?
The email should send only the devices that meet the criteria of the IF statement with the name of the device and the heatingSetpoint.

3) What is happening/not happening?
I am receiving an email with all of the devices, and no heatingSetpoint.

4) Post a Green Snapshot of the piston

5) Attach logs after turning logging level to Full

11/11/2020, 11:20:18 AM +102ms
+0ms ╔Received event [SmithRidge].time = 1605111619285 with a delay of -1184ms
+119ms ║RunTime Analysis CS > 32ms > PS > 72ms > PE > 16ms > CE
+122ms ║Runtime (40313 bytes) successfully initialized in 72ms (v0.3.110.20191009) (121ms)
+123ms ║╔Execution stage started
+131ms ║╚Execution stage complete. (9ms)
+133ms ║Setting up scheduled job for Wed, Nov 11 2020 @ 11:25:19 AM EST (in 301.051s)
+141ms ╚Event processed successfully (141ms)
11/11/2020, 11:15:19 AM +428ms
+1ms ╔Received event [SmithRidge].time = 1605111319285 with a delay of 143ms
+94ms ║RunTime Analysis CS > 27ms > PS > 52ms > PE > 15ms > CE
+96ms ║Runtime (40310 bytes) successfully initialized in 52ms (v0.3.110.20191009) (94ms)
+97ms ║╔Execution stage started
+106ms ║╚Execution stage complete. (9ms)
+107ms ║Setting up scheduled job for Wed, Nov 11 2020 @ 11:20:19 AM EST (in 299.75s)
+113ms ╚Event processed successfully (113ms)
11/11/2020, 11:14:50 AM +17ms
+0ms ╔Received event [SmithRidge].test = 1605111290016 with a delay of 0ms
+119ms ║RunTime Analysis CS > 22ms > PS > 62ms > PE > 36ms > CE
+122ms ║Runtime (40309 bytes) successfully initialized in 62ms (v0.3.110.20191009) (120ms)
+123ms ║╔Execution stage started
+205ms ║║Comparison (decimal) 68.0 is_greater_than_or_equal_to (integer) 67 = true (2ms)
+208ms ║║Comparison (decimal) 68.0 is_greater_than_or_equal_to (integer) 67 = true (1ms)
+210ms ║║Comparison (decimal) 68.0 is_greater_than_or_equal_to (integer) 67 = true (1ms)
+213ms ║║Comparison (decimal) 68.0 is_greater_than_or_equal_to (integer) 67 = true (1ms)
+215ms ║║Comparison (decimal) 68.0 is_greater_than_or_equal_to (integer) 67 = true (1ms)
+223ms ║║Condition #2 evaluated true (81ms)
+224ms ║║Condition group #1 evaluated true (state did not change) (83ms)
+227ms ║║Cancelling statement #5's schedules...
+238ms ║║Calculating (string) Empty device list + (string) - >> (string) Empty device list -
+244ms ║║Calculating (string) Empty device list - + (string) Smith HW Tstat-Bd 1, Smith HW Tstat-Bd 3, Smith HW Tstat-Bd 7, Smith HW Tstat-Bd 8 and Smith HW Tstat-Dng rm >> (string) Empty device list - Smith HW Tstat-Bd 1, Smith HW Tstat-Bd 3, Smith HW Tstat-Bd 7, Smith HW Tstat-Bd 8 and Smith HW Tstat-Dng rm
+4006ms ║║Executed virtual command sendEmail (3759ms)
+4028ms ║╚Execution stage complete. (3905ms)
+4032ms ║Setting up scheduled job for Wed, Nov 11 2020 @ 11:15:19 AM EST (in 25.237s)
+4040ms ╚Event processed successfully (4040ms)

[fieldsjm]

I imported your piston to review more closely. Not to over state the obvious but I’m assuming you moved the IF out of the Every block so that you could test.

The email is sending the names of all the devices without the temperature

In order to include temperature, you will have to adjust the coding and your expression within the email task. I would expect the current expression devicestoCheck + ' - ' + devicesHigh to return only the two lists of devices.

I tweaked the piston a bit
-Updated the devicesHigh to a string variable
-devicesHigh is cleared prior to each run
-each device is individually evaluated, if greater than the defined setpoint, it is added to the devicesHigh list in the format of device1 : device1’s setpoint, device2 : device2’s setpoint, etc
-As long the deviceHigh list is not blank, then the email will be sent with the deviceHigh variable as the body.

[ldale15]

Thanks so much! We are much closer! Now I am getting the names of all of the devices and the temp of the 1 tstat I know is above the IF statement.

[ldale15]

Hey there, just wondering if you have any other suggestions? It seems I am getting the name of every thermostat for the 1 tstat that is above the IF statement. Thanks so much!

[fieldsjm]

hmm, unfortunately I only have 1 thermostat so testing is a little difficult. Do you have any logs to share?

For testing purposes, I replaced heatingSetpoint with temperature, so in theory it would send me all devices with actual temp > 67. The test worked as designed and I let it execute twice to make sure the variables were being set correctly. (Also changed email to a push message)

Are all your thermostats using the same device handler? If not, I am wondering if heatingSetpoint is called something else in the other thermostats.

[ldale15]

Yes, same device handler. here is the log - Thanks!

11/16/2020, 4:50:53 PM +277ms
+0ms ╔Received event [SmithRidge].test = 1605563453276 with a delay of 1ms
+111ms ║RunTime Analysis CS > 17ms > PS > 62ms > PE > 31ms > CE
+114ms ║Runtime (42072 bytes) successfully initialized in 62ms (v0.3.110.20191009) (112ms)
+115ms ║╔Execution stage started
+139ms ║╚Execution stage complete. (25ms)
+141ms ║Setting up scheduled job for Mon, Nov 16 2020 @ 4:55:51 PM EST (in 297.98s)
+150ms ╚Event processed successfully (150ms)
11/16/2020, 4:50:51 PM +39ms
+1ms ╔Starting piston… (v0.3.110.20191009)
+186ms ║╔Subscribing to devices…
+329ms ║║Subscribing to Smith HW Tstat-Bd 1…
+331ms ║║Subscribing to Smith HW Tstat-Bd 3…
+332ms ║║Subscribing to Smith HW Tstat-Bd 7…
+333ms ║║Subscribing to Smith HW Tstat-Bd 8…
+334ms ║║Subscribing to Smith HW Tstat-Dng rm…
+335ms ║╚Finished subscribing (151ms)
+399ms ║Comparison (dynamic) Empty device list is_greater_than_or_equal_to (integer) 67 = true (2ms)
+411ms ║Comparison (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68, is_not (dynamic) null = true (1ms)
+421ms ║Setting up scheduled job for Mon, Nov 16 2020 @ 4:55:51 PM EST (in 299.938s)
+452ms ╚Piston successfully started (450ms)

[fieldsjm]

Looks like that is a log after manually executing via test button. Can you post a log after the piston executes itself at the 5min interval.

[ldale15]

Ooops, sorry about that! Here you go -

11/18/2020, 8:52:57 AM +600ms
+1ms ╔Received event [SmithRidge].time = 1605707578449 with a delay of -849ms
+149ms ║RunTime Analysis CS > 41ms > PS > 74ms > PE > 34ms > CE
+152ms ║Runtime (42075 bytes) successfully initialized in 74ms (v0.3.110.20191009) (150ms)
+153ms ║╔Execution stage started
+163ms ║║Cancelling statement #14's schedules...
+168ms ║║Executed virtual command setVariable (2ms)
+176ms ║║Cancelling statement #5's schedules...
+185ms ║║Calculating (string) Smith HW Tstat-Bd 1 + (string) : >> (string) Smith HW Tstat-Bd 1:
+210ms ║║Calculating (string) Smith HW Tstat-Bd 1: + (string) 68 >> (string) Smith HW Tstat-Bd 1:68
+214ms ║║Smith HW Tstat-Bd 1:68
+215ms ║║Executed virtual command log (2ms)
+238ms ║║Comparison (dynamic) 68.0 is_greater_than_or_equal_to (integer) 67 = true (1ms)
+240ms ║║Condition #2 evaluated true (21ms)
+241ms ║║Condition group #1 evaluated true (state did not change) (23ms)
+244ms ║║Cancelling statement #18's schedules...
+256ms ║║Calculating (string) + (string) Smith HW Tstat-Bd 1 >> (string) Smith HW Tstat-Bd 1
+261ms ║║Calculating (string) Smith HW Tstat-Bd 1 + (string) : >> (string) Smith HW Tstat-Bd 1 :
+279ms ║║Calculating (string) Smith HW Tstat-Bd 1 : + (string) 68 >> (string) Smith HW Tstat-Bd 1 : 68
+283ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68 + (string) , >> (string) Smith HW Tstat-Bd 1 : 68,
+290ms ║║Executed virtual command setVariable (4ms)
+301ms ║║Cancelling statement #5's schedules...
+311ms ║║Calculating (string) Smith HW Tstat-Bd 3 + (string) : >> (string) Smith HW Tstat-Bd 3:
+338ms ║║Calculating (string) Smith HW Tstat-Bd 3: + (string) 68 >> (string) Smith HW Tstat-Bd 3:68
+344ms ║║Smith HW Tstat-Bd 3:68
+346ms ║║Executed virtual command log (3ms)
+372ms ║║Comparison (dynamic) 68.0 is_greater_than_or_equal_to (integer) 67 = true (2ms)
+373ms ║║Condition #2 evaluated true (23ms)
+375ms ║║Condition group #1 evaluated true (state did not change) (24ms)
+378ms ║║Cancelling statement #18's schedules...
+388ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68, + (string) Smith HW Tstat-Bd 3 >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3
+392ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 + (string) : >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 :
+412ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : + (string) 68 >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68
+417ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68 + (string) , >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,
+422ms ║║Executed virtual command setVariable (3ms)
+432ms ║║Cancelling statement #5's schedules...
+442ms ║║Calculating (string) Smith HW Tstat-Bd 7 + (string) : >> (string) Smith HW Tstat-Bd 7:
+466ms ║║Calculating (string) Smith HW Tstat-Bd 7: + (string) 68 >> (string) Smith HW Tstat-Bd 7:68
+472ms ║║Smith HW Tstat-Bd 7:68
+473ms ║║Executed virtual command log (2ms)
+500ms ║║Comparison (dynamic) 68.0 is_greater_than_or_equal_to (integer) 67 = true (2ms)
+502ms ║║Condition #2 evaluated true (25ms)
+504ms ║║Condition group #1 evaluated true (state did not change) (26ms)
+507ms ║║Cancelling statement #18's schedules...
+520ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68, + (string) Smith HW Tstat-Bd 7 >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7
+525ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 + (string) : >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 :
+543ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : + (string) 68 >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68
+546ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68 + (string) , >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,
+552ms ║║Executed virtual command setVariable (3ms)
+560ms ║║Cancelling statement #5's schedules...
+569ms ║║Calculating (string) Smith HW Tstat-Bd 8 + (string) : >> (string) Smith HW Tstat-Bd 8:
+590ms ║║Calculating (string) Smith HW Tstat-Bd 8: + (string) 68 >> (string) Smith HW Tstat-Bd 8:68
+594ms ║║Smith HW Tstat-Bd 8:68
+595ms ║║Executed virtual command log (2ms)
+616ms ║║Comparison (dynamic) 68.0 is_greater_than_or_equal_to (integer) 67 = true (1ms)
+618ms ║║Condition #2 evaluated true (20ms)
+619ms ║║Condition group #1 evaluated true (state did not change) (21ms)
+621ms ║║Cancelling statement #18's schedules...
+630ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68, + (string) Smith HW Tstat-Bd 8 >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8
+634ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 + (string) : >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 :
+648ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : + (string) 68 >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68
+651ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68 + (string) , >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,
+657ms ║║Executed virtual command setVariable (4ms)
+666ms ║║Cancelling statement #5's schedules...
+674ms ║║Calculating (string) Smith HW Tstat-Dng rm + (string) : >> (string) Smith HW Tstat-Dng rm:
+696ms ║║Calculating (string) Smith HW Tstat-Dng rm: + (string) 68 >> (string) Smith HW Tstat-Dng rm:68
+700ms ║║Smith HW Tstat-Dng rm:68
+701ms ║║Executed virtual command log (2ms)
+722ms ║║Comparison (dynamic) 68.0 is_greater_than_or_equal_to (integer) 67 = true (1ms)
+724ms ║║Condition #2 evaluated true (20ms)
+725ms ║║Condition group #1 evaluated true (state did not change) (21ms)
+728ms ║║Cancelling statement #18's schedules...
+738ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68, + (string) Smith HW Tstat-Dng rm >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm
+741ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm + (string) : >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm :
+759ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : + (string) 68 >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68
+762ms ║║Calculating (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68 + (string) , >> (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68,
+768ms ║║Executed virtual command setVariable (4ms)
+776ms ║║Comparison (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68, is_not (dynamic) null = true (1ms)
+778ms ║║Condition #13 evaluated true (5ms)
+779ms ║║Condition group #12 evaluated true (state did not change) (7ms)
+781ms ║║Cancelling statement #16's schedules...
+792ms ║║Calculating (string) These devices are out of range: + (string) Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68, >> (string) These devices are out of range: Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68,
+795ms ║║Calculating (string) These devices are out of range: Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68, + (string) as of >> (string) These devices are out of range: Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68, as of
+799ms ║║Calculating (string) These devices are out of range: Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68, as of + (string) Wed, Nov 18 2020 @ 8:52:58 AM EST >> (string) These devices are out of range: Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68, as of Wed, Nov 18 2020 @ 8:52:58 AM EST
+802ms ║║Calculating (string) These devices are out of range: Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68, as of Wed, Nov 18 2020 @ 8:52:58 AM EST + (string) >> (string) These devices are out of range: Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68, as of Wed, Nov 18 2020 @ 8:52:58 AM EST
+804ms ║║Executed virtual command setState (1ms)
+1861ms ║║Executed virtual command sendEmail (1052ms)
+1864ms ║║Cancelling statement #20's schedules...
+1870ms ║╚Execution stage complete. (1717ms)
+1884ms ║Setting up scheduled job for Wed, Nov 18 2020 @ 8:57:58 AM EST (in 298.966s)
+1892ms ╚Event processed successfully (1892ms)

[fieldsjm]

Thanks, so maybe I am confused as to what is happening vs what you expect. According to the logs, the pistons is evaluating each thermostat as expected. The resulting string states 68 as the set point for each because that is what the piston sees for each device (it is not reading only the first). The final string is:

Smith HW Tstat-Bd 1 : 68,Smith HW Tstat-Bd 3 : 68,Smith HW Tstat-Bd 7 : 68,Smith HW Tstat-Bd 8 : 68,Smith HW Tstat-Dng rm : 68

If they are not all set to 68 then that is a different problem.

[ldale15]

So only 1 of 5 tstats is set to 68. The others are set to all different temps. So technically only 1 should show. The goal is to show tstats that are set over 68.

[fieldsjm]

Just as a test, I’d like to see what ST is reporting the set point to be…in the evaluation console, try pulling the set point for one of the other thermostats

for example:
[Smith HW Tstat-Bd 3 : heatingSetpoint]

[ldale15]

BINGO - (expression) [Smith HW Tstat-Bd 3 : heatingSetpoint] »»» (decimal) 68. Although, I shouldn’t be so happy, now I have to figure out on the ST side how to make it play nice. Thanks so much for your help!!!

[fieldsjm]

now I have to figure out on the ST side how to make it play nice

Good luck

[ldale15]

All set. Thanks again!!!