Only one inverter data is counted for GW_STATION_ID

[markruys]

Only one inverter data is counted for GW_STATION_ID

[eNeRGy164]

As a note, when changing to the new API, I saw it would be easy to iterate through the list of invertors. Also removing the required parameter for the application. So it shouldn't be hard to add this to the script.
At that moment I didn't want to change that part of the script as it was not related to supporting the new API.

[vijghph]

It looks rather easy to do indeed:
Just a loop to add at lines 33-43 of https://github.com/markruys/gw2pvo/blob/master/gw2pvo/gw_api.py

I guess it should be something like:
for inverterData in data['inverter']
then in the loop summing up energy metering, and averaging voltage.

Unfortunately, I never programmed in Python and don't know the build lifecycle. So I would prefer not to create troubles in the code. I hope there will be some volunteer, @eNeRGy164 :)

I am volunteer ...for testing it.

May we register this in the backlog of desired enhancements for the tools?

[vijghph]

I just proposed a file changed that I have properly tested.

Can you validate it to get it approved.
I hope it would eventually go also to the Docker image.

[vijghph]

Any hope to get a build with the proposed fix?

[vijghph]

Was it included in today build? If not, what should I do to get it released?

[wiljums]

From which inverter is the data uploaded currently? And is it possible to get the data of the other inverter? I have currently only offline readings, which could come from my old (offline) inverter. The data of the new one isn't uploaded..

[markruys]

I close this ticket as it's a duplicate of #19. In #19 I've added a reference to a feature branch which I'll merge if someone with multiple inverters can test the change.

[markruys]

I was under the impression that the request for support for multiple strings and multiple inverters were basically the same. Which is not, so I reopen this ticket, as this one is about multiple inverters.

I've created a feature branch which should do the trick. I have only a single inverter, so can someone test this change?

[bramhut]

I would be able to test it, however I'm not sure how to install the feature branch. Can you give me some hints maybe?
I'm still on version 1.2.1 and running it in docker on Windows 10.

[markruys]

But I've merged this feature into trunk and released it as 1.3.4. You can try it like:

sudo pip3 install gw2pvo==1.3.4

[bramhut]

Okay thanks, it works good now!