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 BMS "Failed setup, will retry: device communication timed out" since v1.7.0 #89

Closed
5 tasks done
flibble2000 opened this issue Nov 17, 2024 · 5 comments · Fixed by #92
Closed
5 tasks done
Assignees
Labels
bug Something isn't working

Comments

@flibble2000
Copy link

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:

Daly BMS

Describe the issue

Hi,

Thank you for the integration, it has been incredibly useful.

Since v1.7 I am unable to connect to my Dalys BMS. Rolling back to v1.6.4 fixes the problem.

I am using the BT proxy on ESPHome 2024.10.3 and BT signal strength is -48db. The same BT proxy is used for 2 Victron devices, I've disabled the Victron integration, but it's not helped.

Personally, I'm not bothered as I'll continue with v1.6.4. But hopefully this'll help for future builds if it's a fault.

Reproduction steps

1. Upgrade to any version v1.7.0 onwards
2. 
3.
...

Debug logs

esphome.log
home-assistant_bms_ble_2024-11-17T09-13-35.360Z.log

@flibble2000 flibble2000 added the bug Something isn't working label Nov 17, 2024
@patman15 patman15 self-assigned this Nov 17, 2024
@patman15
Copy link
Owner

@flibble2000 thanks for raising this and providing a log! I suspect it has (again) to do with a previous request to add the MOS temperature value which is not present on all BMS of this type. Could you please try the modified code on the branch (see column on the right "Development") by following the instructions for manual installation? If you could provide a log again that would be awesome!

@flibble2000
Copy link
Author

Nice one! Thanks, give me a day or two and I'll report back.

@flibble2000
Copy link
Author

Hi,

That has fixed it, logs attached.

home-assistant_bms_ble_2024-11-18T11-06-34.729Z.log

@patman15
Copy link
Owner

Great, thanks for the feedback! I'm sorry for the regression, but there seem to be 3 types of Daly BMS

  1. with MOS temperature sensor
  2. without but responding to the request
  3. without and not responding to the request

On request, I added the sensor and now you unfortunately found the 3rd type. I will fix it with the next release in a clean way. Thanks for being patient and reporting the issue! Without having such a battery there is no chance to figure that out in advance since specifications are practically non-existent.

@flibble2000
Copy link
Author

Not a problem, I'm glad I could assist. Thank you for all you hard work.

@patman15 patman15 linked a pull request Nov 18, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants