search

thomascys / solaxcloud

SolaxCloud integration for Home Assistant.
22 stars 11 forks source link

Adding decimal values to yield data #6

Open superyupkent opened 4 years ago

superyupkent commented 4 years ago

Hi Thomas,

Thanks for your work on the Solax software. I have it installed since a few days but I noticed I cant get data out with the same decial values as I see in the solax monitoring cloud. Two other small things I noticed, the unit of measurement is 'kW' instead of 'kWh' and the scraping value does not tend to go below 5 minutes even if you set it to a lower value.

Thanks,

Alex van der Baan

sm0keKings commented 4 years ago

Hi Alex, I have done some work on this and have some answers for you. Although they may not be helpfully since i have noticed they have changed the Solax Portal today to have a API Key and different outputs for the API. This change will help us all in the longrun.

The 5 minute issue seems to be the limit and I think Thomascys mentioned they will block you if you keep logging in to often.

To fix the decimal problem you need to modify these values.

/config/custom_components/solaxcloud/sensor.py

image

I will open a issue with Thomascyc to see if he can rewrite the code and upload the extra work I have done if he needs it to fix the reporting. Understanding he has done all this in his spare time we may have to wait till he has time to re write it.

pieterzen commented 4 years ago

I'm working on an updated version using the realtime API. I'm resolving this issue by returning the value in W, not kW; it matches other energy sensors better. And it'll be WattHour, not Watt.