homeassistant-grott icon indicating copy to clipboard operation
homeassistant-grott copied to clipboard

Incorrect unit on divider for SPF

Open goandfix opened this issue 2 years ago • 6 comments

Hi I'm not sure if it happens with the other units of gorwatt but I have SPF3000 and the unit to meassure in the all current section I had to change it per divider 10 instead 100 and into the load percentage I changed it from 1 to 10. Thanks!

goandfix avatar Nov 07 '23 19:11 goandfix

Hi @goandfix - Can you please try running from the main version of the integration. I believe I've already fixed this for SPF systems, I just haven't done the formal release yet.

muppet3000 avatar Nov 07 '23 21:11 muppet3000

Nop it's still wrong:

sensorfix

In the left side is your file hosted in the github and in the right side is the value that I already fixed on my local installation.

goandfix avatar Nov 08 '23 19:11 goandfix

Here's the file with changes. sensors_mqtt.zip

goandfix avatar Nov 08 '23 19:11 goandfix

Thanks @goandfix - is it just those two that you've highlighted, I didn't realise you were referring to 'current' as in the power unit, I thought you meant that the current version on main was all wrong. Thanks for the zip, I'll do a diff, but if it's just the changes in the screengrab I can change those no problem.

muppet3000 avatar Nov 09 '23 20:11 muppet3000

It's my pleaseure my friend to help you!

goandfix avatar Nov 10 '23 20:11 goandfix

I've just pushed a fix for the current dividers to the 'main' branch - could you try switching to the 'main' release of the integration and testing it works correctly? I'll close this when I do a release with the fix in it.

muppet3000 avatar Nov 10 '23 21:11 muppet3000