search

dylian94 / domoticz-GoodWeSEMS

Domoticz GoodWe Solar Inverter plugin (SEMS API)
http://www.domoticz.com/forum/viewtopic.php?f=65&t=28983
MIT License
12 stars 12 forks source link

Add exceptions #9

Open JanJaapKo opened 4 years ago

JanJaapKo commented 4 years ago

This is an additional PR on top of #5 to add exception handling, to catch scenario's of no token, and no device list

djoenez commented 4 years ago

code running without errors for 4 days.

JanJaapKo commented 4 years ago

@djoenez I presume this also implements #8 ?

djoenez commented 4 years ago

Yes, closing the issue!

djoenez commented 4 years ago

i sometimes see flatlines, no errors in the log. When i check the readings in the SEMS app they are not flat, do you maybe have an idea where this is coming from?

image

Fnasy commented 4 years ago

Flatlines are coming from SEMS app. After a while they are corrected but too late for Domiticz which got data already. Sometime it needs more then one hour to be adjusted in SEMS.

Le February 10, 2020 à 8:26 PM djoenez <http://notifications@github.com > a écrit:

i sometimes see flatlines, no errors in the log. When i check the readings in the SEMS app they are not flat, do you maybe have an idea where this is coming from?

[image] https://user-images.githubusercontent.com/47869328/74153525-1c3aec00-4c11-11ea-9721-73170b03d749.png

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/dylian94/domoticz-GoodWeSEMS/pull/9?email_source=notifications&email_token=AM7NSVRD4LUWCPCLX7RZPTLRCFIW5A5CNFSM4KK5CY42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELIPWCA#issuecomment-584121096 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AM7NSVWSG6JJPOMIOJV5P7TRCFIW5ANCNFSM4KK5CY4Q .
JanJaapKo commented 4 years ago

That last commit fixes issue #10