Whole number issue back after update to 3.5.2

hultenvp / solis-sensor

HomeAssistant integration for the SolisCloud PV Monitoring portal via SolisCloud API

Apache License 2.0

207 stars 42 forks source link

Whole number issue back after update to 3.5.2 #367

Open deligatedgeek opened 5 months ago

deligatedgeek commented 5 months ago

Probably another Solis API "update", but after upgrading to 3.5.2 only whole numbers are showing in home assistant. See the change on the graph below. Screenshot 2024-04-17 at 10 22 29

Same effect as previous Solis change Screenshot 2024-04-17 at 10 37 22

Versions HA Version: Docker 2024.4 HACS version: 1.34.0 Integration version: 3.5.2

deligatedgeek commented 5 months ago

I think the code should be reverted and the misspelled inveterDetail endpoint should be used. As that endpoint is documented here

hultenvp commented 1 month ago

Hi @deligatedgeek,

Picking up again. Is this problem still there or did they fix the resolution in the meantime? If it's still there then I need to think hard how to proceed as there are some conflicting issues and I do not want to go into cherrypicking data from various endpoints trying to make right what's made wrong by Ginlong Solis. (see https://github.com/hultenvp/solis-sensor/issues/360#issuecomment-2028799757)

jmccomb1979 commented 2 weeks ago

I think this was 'fixed' by Ginlong Solis, although I've noticed over the past few days, reported battery charging/discharging is back to whole numbers, with the fraction left over getting caught up in the 23:00-00:00 period. This might be due to the Home Assistant 2024.9 update, as graphs before 4th September do not show the whole number jumps..