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

JBD - DC House battery not detected #157

Closed
5 tasks done
mbeeuid opened this issue Jan 26, 2025 · 6 comments · Fixed by #180
Closed
5 tasks done

JBD - DC House battery not detected #157

mbeeuid opened this issue Jan 26, 2025 · 6 comments · Fixed by #180
Assignees
Labels
enhancement New feature or request

Comments

@mbeeuid
Copy link

mbeeuid commented Jan 26, 2025

Checklist

  • I have tried the latest version of BMS_BLE-HA.
  • I have enabled debug logging for my installation.
  • This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
  • This issue is not a duplicate issue of any previous issue.
  • I have filled out the issue template to the best of my ability.

Provide the type of BMS you used:

DC House: DBJBD (JBD) Firmware ver: 6.6 Device: DP04S007L4S120A

Describe the issue

DC House battery not discovered even though it appears to use a JBD BMS. I see a new "DCHouse" Branch, will this cover this particular BMS? (155-new-integration-dc-house-12v-lifepo4-battery-with-bluetooth)

Reproduction steps

1.
2.
3.
...

Debug logs

Battery not found. (no log)

@mbeeuid mbeeuid added the bug Something isn't working label Jan 26, 2025
@patman15 patman15 added enhancement New feature or request and removed bug Something isn't working labels Jan 26, 2025
@patman15
Copy link
Owner

Device A5:C2:37:2E:A1:17 (public)         
        Name: DP04S007L4S120A                          
        Alias: DP04S007L4S120A                         
        Paired: no                                     
        Bonded: no                        
        Trusted: no                                    
        Blocked: no                                    
        Connected: no                                  
        LegacyPairing: no                              
        UUID: Generic Access Profile    (00001800-0000-1000-8000-00805f9b34fb)
        UUID: Generic Attribute Profile (00001801-0000-1000-8000-00805f9b34fb)
        UUID: Device Information        (0000180a-0000-1000-8000-00805f9b34fb)
        UUID: Unknown                   (0000ff00-0000-1000-8000-00805f9b34fb)
        UUID: Vendor specific           (00010203-0405-0607-0809-0a0b0c0d1912)
        Modalias: usb:v248Ap8266d0001                             
        ManufacturerData.Key: 0xa117 (41239)                      
        ManufacturerData.Value:                              
  2e 37 c2 a5                                      .7..
        AdvertisingFlags:                                         
  06                   

@patman15
Copy link
Owner

I see a new "DCHouse" Branch, will this cover this particular BMS? (155-new-integration-dc-house-12v-lifepo4-battery-with-bluetooth)

Yeah, I yesterday started working on it but did not come far. Please do not open new issues for the same topic. I will now switch over to this issue and add the necessary changes in a branch which is also linked to this ticket.

@patman15 patman15 self-assigned this Jan 26, 2025
@patman15
Copy link
Owner

@mbeeuid please try the linked branch and provide me a debug log, so I can verify the detection and add matching tests for the future.
Thanks, Patrick

@mbeeuid
Copy link
Author

mbeeuid commented Jan 26, 2025

Sorry Patman, I just got called out of the country and will not be able to assist until next week. Just sent you some "tea money" though for the prompt attention! Thanks again!

@patman15 patman15 linked a pull request Jan 26, 2025 that will close this issue
@patman15
Copy link
Owner

Thanks, I'll take some tea till you are back. 😃 Let me know if you are ok then I'll merge the changes to the next release.

@mbeeuid
Copy link
Author

mbeeuid commented Feb 10, 2025

V1.11.7 fixed my detection. Batteries are now connected to HA! Thank you very much, more tea on the way!

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
2 participants