search

brianegge / upnp-ups-service

Service to connect SmartThings to a UPS
MIT License
3 stars 0 forks source link

doesn't seem to be updating #4

Open brianegge opened 5 years ago

brianegge commented 5 years ago
  1. This doesn't seem to be updating unless I hit the manual refresh inside the SmartApp. Is there a place to adjust a polling interval?
brianegge commented 5 years ago

I was originally hoping the poll model would work fine. In the smart app I am requesting frequent polling, but in the logs I only see it working occasionally. The event model is registering a callback, but I havn't implemented sending the callbacks in the python code. I'll take a crack at it next week, if you @kahrbomb are able to test it.

kahrbomb commented 5 years ago

I can definitely test it for you! Let me know and I will jump on it. Right now I appear to get no update unless I manually refresh it. This stinks because I already have my WebCore piston telling me when the UPS switches between battery and line input.

On Sun, Apr 14, 2019 at 1:33 PM Brian Egge notifications@github.com wrote:

I was originally hoping the poll model would work fine. In the smart app I am requesting frequent polling, but in the logs I only see it working occasionally. The event model is registering a callback, but I havn't implemented sending the callbacks in the python code. I'll take a crack at it next week, if you @kahrbomb https://github.com/kahrbomb are able to test it.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/brianegge/upnp-ups-service/issues/4#issuecomment-483024389, or mute the thread https://github.com/notifications/unsubscribe-auth/AcdbFywkJdaf_8wWngiY3FqO1AOF451Iks5vg2Z0gaJpZM4cuvdy .

-- Chris Lindsey

riccanet commented 5 years ago

Any update on this? Willing to test as well.

brianegge commented 5 years ago

I've updated it. I'm getting alerts but haven't figured out what alert to publish in smart home monitor. Also, updated the DHT.