Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong values due to Communication problems #184

Open
k2dp2k opened this issue Sep 11, 2023 · 2 comments
Open

Wrong values due to Communication problems #184

k2dp2k opened this issue Sep 11, 2023 · 2 comments
Labels
solarman Related to solarman

Comments

@k2dp2k
Copy link

k2dp2k commented Sep 11, 2023

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... StephanJoubert/home_assistant_solarman#270

Is it possible to implement something like that?

@kellerza
Copy link
Owner

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
Copy link
Author

k2dp2k commented Sep 12, 2023

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

@kellerza kellerza added the solarman Related to solarman label Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solarman Related to solarman
Projects
None yet
Development

No branches or pull requests

2 participants