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

Daly 3S and 4S BMS Support #24

Closed
yannickudb opened this issue Jun 28, 2021 · 9 comments · Fixed by #41
Closed

Daly 3S and 4S BMS Support #24

yannickudb opened this issue Jun 28, 2021 · 9 comments · Fixed by #41
Labels
enhancement New feature or request

Comments

@yannickudb
Copy link

3S And 4S BMS have different communications.
My contact at Daly says this should be the document that describes the communication between 3 and 4S BMS and PC.

Daly RS485+UART protocol.pdf

@dreadnought
Copy link

The attached PDF is the same as in #3, but it is for a different protocol. Daly sells BMS with two different chipsets under the same brand. You can read through this discussion if you want to know all the details:
dreadnought/python-daly-bms#1

I didn't find documentation neither, but I've implemented most of what it can by capturing the communication of the Windows tool. Here is the code:
dreadnought/python-daly-bms@8799c7b

@Louisvdw Louisvdw changed the title 3S and 4S BMS Support Daly 3S and 4S BMS Support Jun 30, 2021
@unifiedcommsguy
Copy link

I have a Daly 4s with sinoweath protocol and a VenusOS and happy to assist in testing with this.

I'm just looking at the code now and will see if I can provide some feedback/assistance.

@Louisvdw Louisvdw added the enhancement New feature or request label Jul 20, 2021
@liufuyang
Copy link
Contributor

I have a Daly/MGod 10S BMS and I have some issue using it #36

Perhaps this is related to it? Thanks.

@SanderV4n
Copy link
Contributor

I have a Daly Smart BMS 250A, already connected it to my venus GX, and started on a implementation. For now I can read the first values, still needs some work before I can make a pull request. (need to polish my python skills a bit, the python 2.7 in the venus OS does not help)

For the implementation, there is actually quite a bit information on the protocol in the PDF from the BMS tool installer. This is partly in Chinese, but with translate and the original side by side there is a lot to make out. Even bitmaps for some of the status/config commands are given.
Sinowealth BMS Tool Introduction V0.2.pdf

@SanderV4n
Copy link
Contributor

I just pushed a first version to my repo. It needs some cleanup and fetching a bit more info from the BMS in stead of hardcoded variables, and probably has some bugs. Feel free to test/comment. I guess I will continue the implementation the coming days, and will post a pull request when I feel its ready for a first version.

But hey, it works!
image

@SanderV4n
Copy link
Contributor

First version of the pullrequest is up, so have a look! #41

@Louisvdw Louisvdw linked a pull request Sep 6, 2021 that will close this issue
@Louisvdw
Copy link
Owner

Louisvdw commented Sep 6, 2021

Nicely done @Carstijn
I'll review the pull request as soon as I can.

@Louisvdw
Copy link
Owner

Louisvdw commented Sep 8, 2021

https://github.com/Louisvdw/dbus-serialbattery/releases/tag/v0.7beta2
includes the Daly with Sinowealth chip for those that would like to try it and give feedback

@unifiedcommsguy
Copy link

https://github.com/Louisvdw/dbus-serialbattery/releases/tag/v0.7beta2
includes the Daly with Sinowealth chip for those that would like to try it and give feedback

Thankyou for this. Will test out and let you know this week.

Louisvdw pushed a commit that referenced this issue Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants