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

New inverter protocol: Sungrow CAN #552

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

dalathegreat
Copy link
Owner

@dalathegreat dalathegreat commented Oct 21, 2024

What

This PR implements Sungrow CAN, which emulates a Sungrow SBR064 battery

Why

To increase compatibility with more inverters

How

Massive thanks to Chair_Stealer for the CAN logs, and to mathl® for the reverse engineering! 🙌

@dalathegreat dalathegreat marked this pull request as ready for review October 22, 2024 16:42
@itsallanetwork
Copy link

Please! Im dying to impliment this new Sungrow CAN! 👍

itsallanetwork
itsallanetwork approved these changes Nov 6, 2024
@dalathegreat
Copy link
Owner Author

dalathegreat commented Nov 8, 2024

Hi @itsallanetwork
We wont merge this until it has been tested. Feel free to give it a go if you want!

https://github.com/dalathegreat/Battery-Emulator/archive/refs/heads/feature/sungrow-can.zip

@dalathegreat
Copy link
Owner Author

@sm133 Your inverter supports BYD CAN. We can stop discussing anything in this PR related to your troubleshooting, since you will not be able to use Sungrow CAN.

Please make a new Issue for troubleshooting your install: https://github.com/dalathegreat/Battery-Emulator/issues

I will be deleting all comments from here, since this PR focuses on integrating Sungrow CAN, not on troubleshooting your BYD_CAN install.

Repository owner deleted a comment from sm133 Nov 22, 2024
Repository owner deleted a comment from sm133 Nov 22, 2024
Repository owner deleted a comment from sm133 Nov 22, 2024
Repository owner deleted a comment from sm133 Nov 22, 2024
Repository owner deleted a comment from sm133 Nov 22, 2024
Repository owner deleted a comment from sm133 Nov 22, 2024
@itsallanetwork
Copy link

itsallanetwork commented Dec 9, 2024

Hi @dalathegreat,

I am happy to test this now I have my hardware in place. I am running a Tesla M3 82kwh Battery (penhouse fully intact) with Sungrow SH5.0RS Inverter. I have 10a fuse for capacitor (outside penthouse) and 32a for pos and neg from HV to Inverter.

Battery contacters are proven to work. I have not attempted connecting Inverter yet, but I am ready to do so.

I have setup a single Lilygo, but I read sungrow and leaf battery do not work well together given sungrow's 'bug'. I am not sure if this would be applicable given the M3 Tesla battery has a penthouse that is the brains for the battery, I would think the penthouse would protect the battery anyhow. Though of course I can install a CAN-FD MCP2518 board.

What do you recommend for this test?

@@ -51,6 +51,7 @@
//#define SMA_TRIPOWER_CAN //Enable this line to emulate a "SMA Home Storage battery" over CAN bus
//#define SOFAR_CAN //Enable this line to emulate a "Sofar Energy Storage Inverter High Voltage BMS General Protocol (Extended Frame)" over CAN bus
//#define SOLAX_CAN //Enable this line to emulate a "SolaX Triple Power LFP" over CAN bus
//#define SUNGROW_CAN //Enable this line to emulate a "Sungrow SBR064" over CAN bus
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dalathegreat, please remember to also add this to the GitHub workflows.

Repository owner deleted a comment from itsallanetwork Dec 20, 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

Successfully merging this pull request may close these issues.

3 participants