search

RDols / DomoticzDzVentsScripts

Various DzVents scripts for domoticz is use
MIT License
2 stars 0 forks source link

Issue: to monitor create an dummy device type="Electric (Instant+Counter) name="nil" #1

Closed McKormick closed 3 years ago

McKormick commented 3 years ago

Hi Richard I don't know if you look into this anymore but I faced problem with working of your script since a week. It was working ok till then. Now script returns not existing device name

2021-04-09 15:15:02.174 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.174 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.174 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.175 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.175 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.175 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.175 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.175 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.175 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.175 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.175 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.176 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.176 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.176 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.176 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.176 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.176 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.176 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.177 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.177 Status: dzVents: dzRequestSolarInfo : to monitor create an dummy device type="Electric (Instant+Counter) name="nil" 2021-04-09 15:15:02.177 Status: dzVents: dzRequestSolarInfo : -=[ End ]===================================================== 2021-04-09 15:15:02.179 Status: dzVents: Info: ------ Finished dzRequestSolarInfo

Correct devices names are: Inverter 1 Panel 1.1.1 ... Panel 1.1.19

RDols commented 3 years ago

Hello McKormick,

At the moment I have the script disabled in my own system because of another issue. I did a quick test I have the same error, probably their response changed.

I will try to fix it this weekend.

Thanks for the notification, will send you a message when it's fixed...

RDols commented 3 years ago

Hello @McKormick,

I updated the script. The error is gone now. Unfortianatly the power (Watt) readings also. It has been removed from the interface the script is using.

McKormick commented 3 years ago

Hi Richard First of all thank you for taking effort to update the script. First comment is you hardcoded siteID in line 73 Secondly the script updates String value not Inverter and Panels as before - which Iam not sure was intended but I'll need to check the read outs tomorrow during daytime. My hierarchy is:

Inverter 1 -> String 1.1 -> Panel 1.1.1 ... Panel 1.1.19

It is a pity that current power (WATT) is unavailable since it is the main trigger for other Domoticz scripts. Maybe you can implement request from Overview site there is current power data: https://monitoringapi.solaredge.com/site/"your siteID"/overview.json?api_key="your API key"

Nevertheless Thank you

RDols commented 3 years ago

Whoops, Sloppy. I removed my siteID from the commits...Thanks.

RDols commented 3 years ago

I personally intrested in the per panel power readings from this plugin. for near-realtime power readings of my inverter i use the modbus. Unfortianatly only total readings.

I'm thinking to use the following request to get the power (Watt) readings.

https://monitoringpublic.solaredge.com/solaredge-web/p/publicSystemData?reporterId=[panel id]&type=panel&activeTab=0&fieldId=[siteID]&isPublic=true

Unfortianatly you only get the details of one panel. so for every panel, string and inverter you need to do this query.

If this doesn't work i will use the intended API, but only gets the total.

Note: "panel id" can be found in the "https://monitoring.solaredge.com/solaredge-apigw/api/sites/[siteID]/layout/logical/" response. It's a 9 digit number.

McKormick commented 3 years ago

Hi Richard Thank you for updating script it is working again. Missing WATT reading is different story not part of initial issue, so I think it is ok to close it. Thanks