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

Support of Deye SUN SG05LP1 #265

Open
BoberMod opened this issue Nov 22, 2024 · 7 comments
Open

Support of Deye SUN SG05LP1 #265

BoberMod opened this issue Nov 22, 2024 · 7 comments
Assignees

Comments

@BoberMod
Copy link

BoberMod commented Nov 22, 2024

Is your feature request related to a problem?

I have Deye SUN-6K-SG05LP1-EU 6KW inverter. Currently, it's not listed in supported devices.

Logger firmware: LSW3_32U_5406_1.06.
Logger serial number: 2975*******

Describe the solution you'd like

I want to try the deye_hybrid.yaml config. As I understand it, it's a generic profile and can work with any Deye inverter. But according to message in wiki, I can't just choose deye_hybrid.yaml because if it's wrong, it will overwrite my settings:

Way more dangerous is when you load wrong profile (e.g. deye_hybrid.yaml instead of deye_sg04lp3.yaml). It will mess up with your values...

Is there a read-only mode to try?

Describe alternatives you've considered

I considered the original implementation of this integration by StephanJoubert and integration by kellerza.

@BoberMod BoberMod added the enhancement New feature or request label Nov 22, 2024
@BoberMod BoberMod changed the title Support of Deye SG05LP1 Support of Deye SUN SG05LP1 Nov 22, 2024
@davidrapan
Copy link
Owner

Hi @BoberMod, this warning is for existing users to be careful when they (for any reason) delete and re-adding back their device. So they don't add it with incorrect profile which could populate existing history with nonsensical data.

If you as a new user do add device with profile which works but display incorrect data you can just delete it and remove history of the sensors using Developer tools -> STATISTICS -> Fix Issue on all entities which will delete it.

List with supported devices contains those which are confirmed from users and I'm sure that the integration works for many more which are not listed.

@BoberMod
Copy link
Author

I can confirm that deye_hybrid.yaml works with my inverter. Thank you!
The sensors that I needed for grid/battery usage work fine. However I found that program statuses (charging) are displayed incorrectly:

Inverter menu Configuration from integration
image image

All sensors and configurations in case someone needs this:

Sensors

image
image
image

Configuration

image
image
image
image
image

I don't have attached generator and PV. Inverter connected to JK BMS using CAN.

@davidrapan
Copy link
Owner

Can you show me reading using built-in ACTION in Developer tools of register number 274?

@BoberMod
Copy link
Author

Action: Solarman: Read Holding Registers (Modbus Function Code 3)
Register: 274
Response:

"274": 5

@davidrapan
Copy link
Owner

davidrapan commented Nov 27, 2024

Okay so there are bits for different modes (GM, BU, CH) which are not configurable using standard methods (inverter display). The only info I found about it was for different brand (but obviously the same device/HW):
gmbuch
at https://framerusercontent.com/assets/7QleXjicnZzpXo4yXMZ78pFc8I.pdf

"274": 5

Means that GM mode is selected.

My questions are:

  • Do even Deye inverters utilize this?
  • If so is GM the only option available?

Can you try to set this configuration using HA?

@davidrapan
Copy link
Owner

davidrapan commented Nov 29, 2024

Those commits should fix it. 😉

@davidrapan davidrapan removed the enhancement New feature or request label Dec 2, 2024
@davidrapan
Copy link
Owner

Is it fixed in the latest release?

@davidrapan davidrapan self-assigned this Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants