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

Grid & Load Power values are incorrect after update to 24.12.06_2 #298

Open
tinntbg opened this issue Dec 6, 2024 · 22 comments
Open

Grid & Load Power values are incorrect after update to 24.12.06_2 #298

tinntbg opened this issue Dec 6, 2024 · 22 comments
Assignees
Labels
bug Something isn't working

Comments

@tinntbg
Copy link

tinntbg commented Dec 6, 2024

Description

After updating to the latest release I needed re-add the integration according to the resolution stated in bug: https://github.com/davidrapan/ha-solarman/issues/283

When I did that the values of Load Power and Grid Power where multiplied by 10. In order to fix this I have updated the configuration of the my inverter HYBRID by Deye and decrease the scale for the 2 sensors from 10 to 1.

Can this be changed in future releases?

Reproduction Steps

Update to latest version and re-add integration.

Log

Not applicable.

Version

24.12.06_2

@tinntbg tinntbg added the bug Something isn't working label Dec 6, 2024
@davidrapan
Copy link
Owner

Yes I'm aware of that but thank you for your report. I will do something about it.

Could you please tell me exact model number of your device? (e.g. SUN-12K-SG04LP3)

@tinntbg
Copy link
Author

tinntbg commented Dec 6, 2024

Thanks for the quick response! My device is: Deye SUN-8K-SG01LP1-EU

@davidrapan
Copy link
Owner

Could you also do for me a screenshot like this one? #281 (comment)

@Fennsy
Copy link

Fennsy commented Dec 6, 2024

Hello, I've the same inverter : Deye SUN-8K-SG01LP1-EU with the same problem than @tinntbg and this is my screenshot.
In the screenshot the param Phases is 3 but I have only 1 Phase :/

deye

@davidrapan
Copy link
Owner

davidrapan commented Dec 6, 2024

In the screenshot the param Phases is 3 but I have only 1 Phase :/

Guess your inverter has it incorrectly written in the registers (Nothing to be really worried about as your device was correctly identified as Single Phase Hybrid Inverter, numeric value: 3).

v24.12.06_4:

@Fennsy
Copy link

Fennsy commented Dec 7, 2024

Is this phase problem due to the configuration of the inverter in the integration or is it directly in the inverter ?
In any case I tested the scale problem for consumption and everything is ok
Thanks

@davidrapan
Copy link
Owner

Is this phase problem due to the configuration of the inverter in the integration or is it directly in the inverter ?

That value is read directly from your device.

@tinntbg
Copy link
Author

tinntbg commented Dec 7, 2024

Sorry for the late response. Here is a screenshot of the attributes:
Screenshot 2024-12-07 at 9 18 06 1

Thanks for the quick fix! I have tested as well and it all looks to be working now!

Should I close the ticket?

@davidrapan
Copy link
Owner

You can leave it for a while. 😉

@davidrapan davidrapan changed the title After update to 24.12.06_2 the values of Load Power and Grid Power are incorrect. Grid & Load Power values are incorrect after update to 24.12.06_2 Dec 7, 2024
@davidrapan
Copy link
Owner

Can please someone confirm if is working single phase detection is working in latest release? And I don't mean displayed value in the device sensor's attributes but in the amount of added sensors to the list (= sensors w/ L2 are not present).

@tinntbg
Copy link
Author

tinntbg commented Dec 9, 2024

Hi @davidrapan,

I have just deleted and re-added my inverter in home assistant and from what I can see there are a lot of sensors with L2 names.

So the short answer is that it's not working.

Thanks!

@tinntbg
Copy link
Author

tinntbg commented Dec 9, 2024

If I manually change the number of MPPTs and the number of phases then the values are not created, but the auto detection is not working with release 24.12.08_9

@davidrapan
Copy link
Owner

davidrapan commented Dec 9, 2024

Could you please show me read of register number 8 using services (actions)? Thank you!

@tinntbg
Copy link
Author

tinntbg commented Dec 9, 2024

Is this what you are looking for or something else?
Screenshot 2024-12-09 at 19 52 42

@davidrapan
Copy link
Owner

davidrapan commented Dec 9, 2024

Mh I guess that's the culprit. So it's either not used or 0 also means single phase...

Edit: I'll proly go w/ default = single phase until someone w/ split phase appears and will complain that he does not see his values. 😆

@davidrapan
Copy link
Owner

So what's the current state @tinntbg?

@tinntbg
Copy link
Author

tinntbg commented Dec 10, 2024

Hey @davidrapan,

Just updated to the latest version, deleted and re-added my inverter, but unfortunately, nothing has changed. I still see L2 values and it's still been detected as a 3 phase inverter.

Thanks!

@tinntbg
Copy link
Author

tinntbg commented Dec 10, 2024

Here are all the Auto discovered values:
Screenshot 2024-12-10 at 21 46 07

@davidrapan davidrapan self-assigned this Dec 12, 2024
@ikeare
Copy link

ikeare commented Dec 13, 2024

Same problem on 12k sg01lp1.
Defined as 1 phase, but essentially 3 phase. Also, “scale” for “GRID Power” and “LOAD Power” does not work correctly, I had to manually change the values

@davidrapan
Copy link
Owner

davidrapan commented Dec 13, 2024

Same problem on 12k sg01lp1

This is not enough information.

I had to manually change the values

What do you mean by that? Where?

@ikeare
Copy link

ikeare commented Dec 14, 2024

Same problem on 12k sg01lp1

This is not enough information.

I had to manually change the values

What do you mean by that? Where?

image

After a recent update of the Deye inverter, integer numbers in watts stopped being displayed, that is, it was 1043 watts, it became 104, judging by Deye Cloud - they removed this figure and multiplied the value by 10, their watts are always equal to 10.
In order to fix this I changed the value of "scale: 10" to "Grid Power" and "Load Power" in deye_hybrid.yaml

@davidrapan
Copy link
Owner

davidrapan commented Dec 14, 2024

There is a checkbox for that in configuration under Additional options. 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants