#Fronius
lost dove
Oh i see u r developer 🙂 and my ticket is assigned to u... 😄
plain steeple
Right 😉
Have you looked for a datalogger firmware update (on solarweb)? I've not seen that response with any of the devices I ran tests for.
That is, both of the errors you describe
lost dove
No, seperated errors. The escaping is a different error.
plain steeple
Yes sure.
lost dove
https://imgur.com/a/s5v7Csi <-- No updates for this device
plain steeple
What is this device? Is it a V0 API thing?
lost dove
Yes, it's a v0 api thing https://imgur.com/miKXnuq
plain steeple
Oh dear... ok
lost dove
There u see also the missing unique_id field from the other inverters.
So i'm not really sure if this is a bug... i think this device is simply very old 😄 I hope u like challenges 🙂
lost dove
https://github.com/home-assistant/core/issues/82322 Here it is... EUR/kWh and kg/kWh
plain steeple
Ok. That's awkward... why escape a "k" 🤷
I guess this is best fixed in the pyfronius library.
The no-unique_id thing... 🤔 not sure yet
lost dove
Oh, when fixing in the pyfronius library when i should tell u a bit more about the stange behaviours from the inverters.
When it's night, and the PVPower of the inverters go to 0 i get the BadErrorStatus Exception from the inverters with the message "Transfer Timeout". The Inverters are still listed in the solar_api/GetInverterInfo.cgi api but when i try to get details about the inverter itself i get the"Transfer timeout" error from the library.
Really strange...
plain steeple
That endpoint is cached. I think the other is something like a relay to Modbus - and that's not reachable when turned off
lost dove
That endpoint is cached. I think the other is something like a relay to Modbus -...
Sounds plausible.
plain steeple
The integration should handle that and query less often until it's available again
plain steeple
Yes, it's a v0 api thing https://imgur.com/miKXnuq
Did you do that at night, or were the inverters running?
lost dove
No this is while the inverters were running. Night is here in about 4-5 Hours.
lost dove
The integration *should* handle that and query less often until it's available a...
Yes, i think the integration handle this ok so far... i will check this today evening.
plain steeple
No this is while the inverters were running. Night is here in about 4-5 Hours.
Yeah, I guess we are in the same timezone. But that doesn't mean it's running. Mine is currently at 142 Watts, and that's a 20 kW Symo... ☁️😶🌫️
lost dove
Germany 🙂 u Austria. Should be the same timezone. Inverters are at 155, 193, 76, 137, 147 (watt). I will make pictures when there are appearing as timeouted.
Should i create an issue for the unique-id thing? Or u like first to see how i "fixed" it?
plain steeple
However you like.
lost dove
https://imgur.com/y9PjZhJ (see components/fronius/__init__.py line 171+)
plain steeple
I see. Id use
unique_id = inverter["unique_id"]["value"] or f'inverter_device_index_{inverter["devicee_id"]["value"]}'
To not break existing installations.
lost dove
Yeah, i still learning the architecture of home assistant and about the impact from changes. Your solution makes more sense (and is so smooth) 🙂
Do u add this as patch? Or should i create a issue first?
plain steeple
Im a Little short on time currently. But if you do a PR (and maybe a unittest with your json data) I'm happy to review/merge.
lost dove
Hmm... to be honest... i don't know how to do this... but when u guide me with this... i will fix/add some integrations.
plain steeple
But tbh the unit test game for that Integration is not very friendly 😬
lost dove
I'm happy that i was able to create a dev environment and modify something 😄 That's it so far.
plain steeple
Yeah it's a tight learning curve
lost dove
Yes... but i think when i understand it i can really help u all. At all thanks so far!
plain steeple
Sure. I guess there are lots of guides around. Otherwise just create an issue and have some patience 😉
lost dove
plain steeple
Cool. Nowadays you can just hit . on GitHub too to make changes and open a PR. I think even run tests. But I guess not test it with local things.
lost dove
i started changing the code in my VSCode enviroment before checking out... so i'm a bit nervous to break things completly (because i don't really unterstand where are the files from my vscode are located at the moment... i simply clicked the button in this guide https://developers.home-assistant.io/docs/development_environment
plain steeple
Haha, I never used devcontainer so no idea where these files really go. I guess wherever you cloned the repo to.
lost dove
Now i have the issue with the inverters:
https://imgur.com/a/ECOx28q Info from the api
Get info about inverter 5 results now in the timeout https://imgur.com/MdN7cO6
And the integration overview shows this https://imgur.com/7qL1roS
plain steeple
That's really hard-to-read image quality. Can you upload that as text somewhere? (Eg. in the GitHub issue you can add text files)
lost dove
Later i create a new issue about this topic.
plain steeple
Or use the thing the chatbot here always does. Can't remember the name rn
plain steeple
One more question: are your inverters on current firmware? (Not the datalogger, but the inverters themselves).
I always use Solarweb to check and update (Einstellungen-> Komponenten).