Jimbo.Automates

Hey @bpwwer, I migrated from my original weatherflow to tempest with a new hub, swapped the station id in PG3 config and restarted and it crashes with: 2022-04-09 12:03:01,238 MQTT udi_interface.interface INFO interface:_connect: MQTT Subscribing to topic: udi/pg3/ns/clients/00:0d:b9:52:bf:fc_9 - MID: 1 Result: 0 2022-04-09 12:03:01,256 MQTT udi_interface.interface INFO interface:_subscribe: MQTT Subscribed Succesfully for Message ID: 1 - QoS: (0,) 2022-04-09 12:03:04,211 MainThread udi_interface.interface INFO interface:addNode: Adding node WeatherFlow(controller) [None] 2022-04-09 12:03:05,418 Thread-1 udi_interface ERROR udi_interface:write: Exception in thread 2022-04-09 12:03:05,420 Thread-1 udi_interface ERROR udi_interface:write: Thread-1 2022-04-09 12:03:05,420 Thread-1 udi_interface ERROR udi_interface:write: : 2022-04-09 12:03:05,421 Thread-1 udi_interface ERROR udi_interface:write: Traceback (most recent call last): 2022-04-09 12:03:05,422 Thread-1 udi_interface ERROR udi_interface:write: File "/usr/local/lib/python3.8/threading.py", line 932, in _bootstrap_inner 2022-04-09 12:03:05,426 Thread-1 udi_interface ERROR udi_interface:write: self.run() 2022-04-09 12:03:05,426 Thread-1 udi_interface ERROR udi_interface:write: File "/usr/local/lib/python3.8/threading.py", line 870, in run 2022-04-09 12:03:05,430 Thread-1 udi_interface ERROR udi_interface:write: self._target(*self._args, **self._kwargs) 2022-04-09 12:03:05,431 Thread-1 udi_interface ERROR udi_interface:write: File "/var/polyglot/pg3/ns/00:0d:b9:52:bf:fc_9/nodes/weatherflow.py", line 115, in parameterHandler 2022-04-09 12:03:05,432 Thread-1 udi_interface ERROR udi_interface:write: self.discover(stationList) 2022-04-09 12:03:05,433 Thread-1 udi_interface ERROR udi_interface:write: File "/var/polyglot/pg3/ns/00:0d:b9:52:bf:fc_9/nodes/weatherflow.py", line 462, in discover 2022-04-09 12:03:05,435 Thread-1 udi_interface ERROR udi_interface:write: info = self.query_station(station['id']) 2022-04-09 12:03:05,436 Thread-1 udi_interface ERROR udi_interface:write: File "/var/polyglot/pg3/ns/00:0d:b9:52:bf:fc_9/nodes/weatherflow.py", line 190, in query_station 2022-04-09 12:03:05,438 Thread-1 udi_interface ERROR udi_interface:write: info['units'] = self.query_station_uom(station, rain_id, rain_type) 2022-04-09 12:03:05,438 Thread-1 udi_interface ERROR udi_interface:write: File "/var/polyglot/pg3/ns/00:0d:b9:52:bf:fc_9/nodes/weatherflow.py", line 234, in query_station_uom 2022-04-09 12:03:05,440 Thread-1 udi_interface ERROR udi_interface:write: p_rain = jdata['obs'][0]['precip_accum_local_yesterday'] 2022-04-09 12:03:05,441 Thread-1 udi_interface ERROR udi_interface:write: KeyError 2022-04-09 12:03:05,442 Thread-1 udi_interface ERROR udi_interface:write: : 2022-04-09 12:03:05,442 Thread-1 udi_interface ERROR udi_interface:write: 'precip_accum_local_yesterday'

Haha, yeah IFTTT is dumb, but when that's the only choice, at least it's there [emoji16] Sent from my Pixel 6 Pro using Tapatalk

No, you create 2 applets, but in PG you create one node and set the applet for in and off

Also, while testing you can turn on the option in IFTTT to send you a notification when the automation is run, and watch the log in PG3 with debug level enabled.

In the AC you can test the node by turning it "On", same with a program. Just make sure you close and re-open the AC after the node was added. BTW, don't you ever one to turn the device off? If so, I would create a node called BRSensibo then define an On and Off Event.

There are 2 examples showing the usage, one is the "old" way setting them on the controller: https://github.com/UniversalDevicesInc-PG3/udi-poly-notification/blob/master/README.md#send-sys-short and the other is the new way using one program line on the service node: https://github.com/UniversalDevicesInc-PG3/udi-poly-notification/blob/master/README.md#send-sys-short-with-params You can put any ${sys.node.somenode.name} ${sys.node.somenode.driver} where the somenode is the node address and driver is the node driver that shoes the data you want as documented here https://wiki.universal-devices.com/index.php?title=ISY-994i_Series:EMail_and_Networking_Substitution_Variables

I'm the PG3 UI under the node server log page set logging to debug and restart, and PM me the log package. It makes no sense that nodes page is empty, you should at least see a controller after installing. Sent from my Pixel 6 Pro using Tapatalk

Thanks all. @asbril I added a documentation example similar to what I described to you above https://github.com/UniversalDevicesInc-PG3/udi-poly-notification/blob/master/README.md#send-sys-short-with-params

I just made a minor change and released 3.3.3, I went to PG3 on my production machine, clicked restart and once it connected the version updated to 3.3.3 as expected. Bob also was unable to see the issue on his machine this morning. So anyone still having the issue with an update not showing properly, please describe in detail what you did and what you did not see and start a new thread so @bpwwer can help.

You have mixed together 2 different methods of sending a notification. The How it Works describes these as "messages" and "notify nodes", they are used individually, not together. The first line in your then sets the "message" to 'Bad Weather', then next three lines set params on a 'notify node' then the last line sends a message on the pushover service. The best way to do this now is to create a Custom Content in the Admin Console which can contain the drivers from the NOAA Weather node to tell you the Alert and Wind speed in the message. Then the program will just have one line in the 'Then' section.

Please stop saying the new version does not show in the PG3 UI after restarting. I've already said: The version info is stored in PG3 database so need help from @bpwwer so I'm sure he will look into when he has time. Sent from my Pixel 6 Pro using Tapatalk

Thanks for sending the log, pushed 3.3.2 to fix the issue which happens if you add a Telegram node, but don't give it a name. You should delete it in the config, but at least now the NS won't crash when it happens.

Nice trick, didn't think of that.

Awesome! Yes, I'm converting many of mine and I'm amazed how many can be converted to share a few different Custom Contents! Also, for everyone's info, I added verbiage to the docs: https://github.com/UniversalDevicesInc-PG3/udi-poly-notification/blob/master/README.md#service-nodes

Ya, that's not good, PM or email me the log package. That usually means there was an error prior to that. But, if that's the last warning then you are ok, it will continue to print those every second until it times out after 60 seconds. I probably should print a warning when it's ok, or just don't print warnings unless it times out. It's there because early on PG3 had issues with handlers but its pretty stable now.

Yes already provided that info to @Javi Sent from my Pixel 6 Pro using Tapatalk

That's something @bpwwer will have to answer. Sent from my Pixel 6 Pro using Tapatalk

I had the same error yesterday that I see in your log: 2022-04-03 20:22:46,838 MQTT udi_interface.interface ERROR interface:_connect: MQTT Failed to connect, invalid identifier Which I discussed with @bpwwer but he wasn't sure why that would have happened. Please restart PG3: sudo service pg3 restart

This is fixed in 3.2.2, but there is still an issue which I think will require an update to UD Mobile. I apologize to @Javi that I didn't give him access to this to test with the new editor that Chris added before this was released.

PG3 Notification node server 3.3.1 released See the Release Notes for more information. As mentioned in the release notes, this fixes the problem @brians found above, and sorry @brians you will have to edit your programs again and save them again, if you haven't already switched over to sys short messages.

Very sorry about this, it's not what I mentioned before, it's completely my fault. This changed from Send to Send Message so the programs do have to be edited. Now the question is, do I put it back so others don't break or leave it as is and just notify everyone... I would also suggest everyone move away from those messages and use the Sys Short Messages, and I'm about to release a new version that makes it easier to send them with one line in a program, but need to decide if put Send back to what it was or leave it...

Good catch @brians I didn't think about that possibility, this sounds like an ISY issue that happens when a device definition is modified, so PG updates the device on the ISY which invalidates programs that reference the device. I've talked to @Chris Jahn about this in the past but I'm not sure if we ever resolved it. Was the program yellow, or it would the program just not automatically run, or the program would run but the notification didn't happen? I currently am not using any old user defined short messages any more so I didn't notice this.

Does this happen on sync? Or when you open that node in UD Mobile? Sent from my Pixel 6 Pro using Tapatalk

Yes, probably in a complex cmdp structure I'll send you the nodedef when I get home. Sent from my Pixel 6 Pro using Tapatalk

The version info is stored in PG3 database so need help from @bpwwer Sent from my Pixel 6 Pro using Tapatalk