Whole number issue back after update to 3.5.2

[deligatedgeek]

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.

Same effect as previous Solis change

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

[deligatedgeek]

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

[hultenvp]

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 #360 (comment))

[jmccomb1979]

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..

[jmccomb1979]

After running the soliscloud_test.py it seems that the api is returning whole numbers for InverterDetailList. JSON output attached for InverterDetailList and InverterDetail endpoints.
datatest1.json
datatest2.json

[jmccomb1979]

Have opened a support ticket for the battery charge/discharge whole number issue, and Solis re-enabled the API access that I already had.....

[jmccomb1979]

I got a reply -

"We have received the reply as the inverter DetailList is calculated by the SC platform, and inverterList is reported by the inverter, so the two values are different, and the R&D staff recommends using inverterDetailList."

I have a fix being tested which pulls only the battery charge/discharge data from inverterList, but the big fiery ball is not visible in the sky today so I'm not seeing any battery activity atm. If it works, I'll submit a PR.

[leroyboerefijn]

@jmccomb1979 hope you've seen some sunlight since! Any luck fixing this issue? :)

[jmccomb1979]

@leroyboerefijn Sadly not. My theory was to use only the battery data from the inverterList endpoint, but my python skills were not up to the task. Attempted to re-do the SoloLearn Python Core course that I had completed 6 years ago, but abandoned it when the example code snippets used in their tutorial questions had no indentation. Pointed it out to them and they said "That one isn't supported any more, use the Python Developer course instead". I've not had the time recently to get back to the course.