-
Notifications
You must be signed in to change notification settings - Fork 104
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
Power values jump to 65535 kW and -65509 kW #398
Comments
I have exactly the same problem with DEYE SG05LP3 20K (the new one). I am using Sunsynk/Deye Inverter Add-on (edge/dev) with Version: e6c5d80 |
Can you test with the new edge version that includes #397 ? |
Let's keep this issue open for now. Pls confirm in a day or two if fix is consistent. |
Its not fixed with newest version Up and down 65xxx Watts. I found that: Looks nearly the same problem on Solarman. Probably you can check how they fixed it. |
Yes, I just checked over at https://github.com/davidrapan/ha-solarman/blob/c1e893c1c79e2b969104bf478deca8f2e286f504/custom_components/solarman/inverter_definitions/deye_p3.yaml#L2461-L2485 that they have the same registers are defined here: sunsynk/src/sunsynk/definitions/three_phase_common.py Lines 78 to 80 in bbb7a60
This appears to be the fix they implemented: StephanJoubert/home_assistant_solarman#270 |
Yeah, that can be a fix. |
For your information: I've found out something else that's very strange! Since the solar has stopped coming in because the sun has gone, there have been no more peaks. From about 4 p.m. onwards, you can't see any more peaks. Is it somehow related to this? |
Probably is related, yes. Now that I think of it, I also get spikes when there is solar production. Maybe data gets a bit messed up when many modbus frames come in. For this reason, I also implemented CRC validation as well (in the last PR above), like other projects have. |
Version e37dd6d is up 🙌 Hopefully this does it. 🤞 |
Now it's completely broken. In the logs I get continously:
|
Not for me. I didnt have that error.
Will check it later, when Solar Power comes in. Without solar, i didnt have that peaks |
Are you using solarman connection? I am guessing not (tcp to modbus?) |
Yes, solarman with the original wifi dongle. |
No, i am using cable from modbus with Waveshare RS485 to USB (USB into my Home Assistant mini PC) |
Just found that there is a related issue from a long time ago: #184 This is strange, as when using the pysolarman driver, under the hood we had already been doing these checks I added, yet the issue is there. Looking through the code, I am starting to think that my validations were not implemented on the right place... This does not change my confusion with why the issue persists when using the pysolarman driver, as they should be already there in the recent versions... @kellerza any ideas here? |
My Solar Power starts now, lets see... right now, on low solar power there are no peaks with newest version. Hope that also doesnt comes with more power soon |
I am skeptical. #408 I think we need to revert this PR, as it only affects solarman (pysolarmanv5) driver usage (and overlaps with validations inside it). |
Mine is working just some spikes are there from time to time... Your environment
|
I will do another PR with revert of solarman breaking changes, also will add more validation to reading of data and more debug logs, so it it continues to happen we can hopefully see why. |
|
It looks like that, but if you get to much power (solar) or anything else, than you get peaks |
yes, I get 65k spikes for CT entities |
Running add on version e37dd6d (edge) with recent bugfix.
Connected via pymodbus to mbusd instance running on a separate Raspberry Pi. Still get glitchy values after the latest fix. I get it on the grid power for each of the three phases as well as on the total grid power value. Below I show:
Versions:
Logs for add-on is spammed with this (Repeating):
Other errros in add-on log repeating less frequently.
|
Issue related to
Sunsynk / mbusd Home Assistant Add-On
Describe the issue/bug
Since the last commit, power sensors randomly jump to 65535 kW and -65509 kW.
Expected behavior
Real values should be shown
Your environment
See also #391 (comment)
The text was updated successfully, but these errors were encountered: