patman15 / BMS_BLE-HA

This integration allows to monitor Bluetooth Low Energy (BLE) battery management systems (BMS) from within Home Assistant.
GNU Lesser General Public License v2.1
50 stars 9 forks source link

Seplos v3 not recognize #46

Closed Alexandros89 closed 2 months ago

Alexandros89 commented 2 months ago

Hi I have 2 batteries with the seplos v3 BMS. From my phone I can connect through Bluetooth without any problem. Using the esphome proxy, ha doesn't recognize anything. The esphome proxy works because it see my beacon's phone

patman15 commented 2 months ago

Your description does not really help to exclude potential issues. Can you please provide a debug log of the esphome proxy as well as one of the integration in order to be able to narrow it down?

Alexandros89 commented 2 months ago

Yes I'm sure the phone is disconnected, I go away from batteries and esp home proxy for over 8 hours and home assistant haven't recognize any battery. From seplos app with Bluetooth I can see the batteries without any problem. I attach some pictures Screenshot_2024-09-19-14-51-55-371_seplos.bms.xzh.com.jpg

Screenshot_2024-09-19-14-52-07-099_seplos.bms.xzh.com.jpg

The esphome proxy haven't other Bluetooth card near it. And the distance from them is 50cm

patman15 commented 2 months ago

Ok, now I think I know the issue:

Seplos v3 (show up as SP0…)

Can you please change the following line https://github.com/patman15/BMS_BLE-HA/blob/8e9c1b7c934d6837fb3f93f2032b290c7677ed2d/custom_components/bms_ble/plugins/seplos_bms.py#L108 and this one: https://github.com/patman15/BMS_BLE-HA/blob/8e9c1b7c934d6837fb3f93f2032b290c7677ed2d/custom_components/bms_ble/manifest.json#L25 both to read SP1* and see if that helps? I need to look into how to autodetect Seplos batteries better as the name seems not to follow a pattern.

Alexandros89 commented 2 months ago

Before you answer me I change only the manifest and it doesn't recognize it. After that I change the script now it appears.

patman15 commented 2 months ago

Does it fully work as expected?

Alexandros89 commented 2 months ago

now it works, i see the basic information that is better than nothing

patman15 commented 2 months ago

Ok, thanks for the feedback, I'll patch the detection for the next release. Compared to the screenshots I see the same information, so not sure about your expectations. Thanks again for the confirmation that it works!

patman15 commented 2 months ago

@Alexandros89 sorry, my commit comment automatically closed the issue. If you still need something, feel free to reopen.

Alexandros89 commented 2 months ago

These are the other information that I can see from the app

Screenshot_2024-09-20-19-12-12-858_seplos.bms.xzh.com.jpg

Screenshot_2024-09-20-19-11-57-275_seplos.bms.xzh.com.jpg