search

kellerza / sunsynk

Deye/Sunsynk Inverter Python library and Home Assistant OS Addon
https://kellerza.github.io/sunsynk/
MIT License
209 stars 88 forks source link

Wrong values due to Communication problems #184

Open k2dp2k opened 1 year ago

k2dp2k commented 1 year ago

Hey hoo, I just saw that I got consumption and production values like 45.000w and even higher on my 5kw inverter... I did a little research and found this issue in the solarman git... https://github.com/StephanJoubert/home_assistant_solarman/pull/270

Is it possible to implement something like that?

kellerza commented 1 year ago

Going through the PR, I read that someone nicked code from pysolarman5, got caught and eventually they started using pysolarmanv5

This addon already uses pysolarmanv5 v3.0.0, so it should include the idea behind the link you shared.

What they do in the component you linked to is to reconnect on any error, something not being done today - see solarman.py This was implemented by the author of pysolarmanv5, so guess this would be the expected way to use the library

k2dp2k commented 1 year ago

Okay thanks for the explanation.. Anyway the problem is still present, as a workaround they implemented something like an plausibility check for the measured values...