Skip to content
New issue

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

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

Already on GitHub? # to your account

New Battery: Daly SmartBMS Support #857

Merged
merged 8 commits into from
Feb 12, 2025

Conversation

M4GNV5
Copy link
Contributor

@M4GNV5 M4GNV5 commented Feb 5, 2025

What

This PR implements the Daly SmartBMS RS485 protocol, allowing to talk to and obtain battery data from these systems.

Why

This is yet another LV component. Many DIY 48V batteries use Daly components, since they are cheap.

There are non-smart Daly versions which do not have communication and "smart" versions which come with CAN and RS485. While implementing the CAN communication is simple(r), I opted for the RS485 to keep the lilygo CAN port unused allowing to use it for the very common pylon-lv CAN protocol for the inverter side.
The payloads for both CAN and RS485 are the same, so reusing some of this code for also implementing the CAN communication should be easy if desired.

How

This is the first RS485 battery implementation, so I had to make some additions to the core.

When

While I was planning to use Twizy batteries as storage, in the end we obtained only cells from Twizy packs (no BMS included) (at only 10€/kWh). I had to change plans and bought a Daly BMS for one of the 4 packs.
I will convert this to a regular PR once everything is tested and running with my Deye LV inverter (pylon-lv protocol).

Wiki

Not sure if I can edit the wiki directly?
usual daly bms voltage range: 4S-24S ~= 14V-100V

@M4GNV5 M4GNV5 marked this pull request as draft February 5, 2025 08:18
@dalathegreat
Copy link
Owner

Nice work! I created this Wiki page which can be used for adding info about the Daly SmartBMS https://github.com/dalathegreat/Battery-Emulator/wiki/Battery:-Daly-SmartBMS

@M4GNV5
Copy link
Contributor Author

M4GNV5 commented Feb 10, 2025

I made some successfull tests. The Deye inverter got the daly data from battery emulator and displayed it on the screen. I did not connect power yet to make sure everything works. I actually found some problems with the pylon LV implementation as well and fixed those.
Let me know if you think a separate PR is required for this.

Below some images. The cabling is still quite experimental 😆. I will create better pictures for the wiki once everything is set up.
I will probably do a full test with power on ~wednesday.

photo_2025-02-10_11-11-28

photo_2025-02-10_11-24-47

@dalathegreat dalathegreat marked this pull request as ready for review February 10, 2025 14:42
Repository owner deleted a comment from Alizainharoon Feb 11, 2025
Copy link
Owner

@dalathegreat dalathegreat left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@dalathegreat
Copy link
Owner

Merging this, so we can do smaller improvement PRs if needed!

@dalathegreat dalathegreat merged commit 6413566 into dalathegreat:main Feb 12, 2025
57 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants