Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Missing many data for some of the inverters after upgrading to 0.6.x #191

Open
2 of 6 tasks
upsuper opened this issue Dec 7, 2024 · 11 comments · Fixed by #195
Open
2 of 6 tasks

[BUG] Missing many data for some of the inverters after upgrading to 0.6.x #191

upsuper opened this issue Dec 7, 2024 · 11 comments · Fixed by #195

Comments

@upsuper
Copy link
Contributor

upsuper commented Dec 7, 2024

Describe the bug
After upgrading to 0.6.x, most inverters are working fine (albeit I need to remove some old sensors and update the ID of the new sensors), but there are some inverters lose majority of its sensors. See the comparison below:

Screenshots showing different inverters
Inverter that works normally Inverter that loses majority of the sensors
image image

Account type

  • Installer
  • DIY / DHZ
  • Home Owner (This integration is not (fully) functional with a home owner account!)

Envoy

  • Metered with CTs installed

  • Metered without CTs

  • Standard

  • FW version: D8.2.127

  • Amount of micro inverters connected: 56

Home Assistant

  • Version: 2024.12.1

Additional context

I took a look at some other issues, specially I saw #189 (comment) and also tried to fetch data from ivp/pdm/device_data, and it seems that it only lists 50 inverters, and the inverters lose the sensors can't be found in that data. So it seems like this endpoint doesn't work as reliably as the previous one I suppose?

@upsuper
Copy link
Contributor Author

upsuper commented Dec 7, 2024

And indeed I can found the inverters in the previous ivp/peb/devstatus endpoint but not the new one.

@vincentwolsink
Copy link
Owner

Hi! That is a serious amount of inverters 😁

Can you please provide the output of the ivp/pdm/device_data endpoint?

@catsmanac
Copy link
Contributor

Looks like the tail-end of device_data shows the limit of 50. Device count matches my 24 inverters and 1 q-relay.

    "deviceCount": 25,
    "deviceDataLimit": 50

@upsuper
Copy link
Contributor Author

upsuper commented Dec 7, 2024

Oh, good point. I have 56 inverters + 2 relays so there should be 58 devices in total. Maybe that's over the limit of this endpoint?

@catsmanac
Copy link
Contributor

catsmanac commented Dec 7, 2024

Might be something that can be configured as datasheets state it can communicate with up to 300 inverters.

@upsuper
Copy link
Contributor Author

upsuper commented Dec 7, 2024

Maybe @krbaker can help shed some light on how to use a different limit or do pagination on this endpoint?

@vincentwolsink
Copy link
Owner

For now, what I can do is if there are more than 50 devices, fallback to the devstatus endpoint. This is not ideal though. Better is to find out how to do pagination..

@upsuper
Copy link
Contributor Author

upsuper commented Dec 9, 2024

Sounds good. This is quite unfortunate but it's probably better than lacking a lot of sensors for some of the devices.

@vincentwolsink
Copy link
Owner

Things are now getting a bit too complex for my liking. But I made usage of the old devstatus endpoint configurable for now in #195 .

@vincentwolsink
Copy link
Owner

Keeping this open, since pagination on the new endpoint would still be the preferred solution.

@upsuper
Copy link
Contributor Author

upsuper commented Dec 13, 2024

I can confirm that enabling devstatus endpoint option in configuration and reload the integration makes those information come back, thanks! (It does mean all the new sensors become unavailable but that's okay.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants