-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
Add bluetooth support #52
base: main
Are you sure you want to change the base?
Conversation
|
notifications.txt
Different notification frames / responses:
|
8c223bd
to
b9d5b76
Compare
d8bcdb7
to
e65ef7a
Compare
Hy Syssi, have been trying to get the bluetooth to work for the last few days, but unfortunately can't get anything out ... any idea what I can do to make it pull the data? [14:08:23][D][esp-idf:000]: W (16767) BT_HCI: hcif disc complete: hdl 0x0, rsn 0x8 |
@manznOnly thanks for poking around. This feature isn't ready yet but your logs look promising already. Just the requests aren't implemented yet so we are sending garbage at the moment:
|
I understand, if you need someone to test - let me know - i have 6 Packs for testing :D |
@syssi Here are the bluetooth specs for 10E. You could put them into the doc section. |
82d677d
to
2613155
Compare
@FritsOV Added. Thanks for your support! |
any update on this if I may ask? Does BT work with the Seplos BMS and ESPHome? |
Hi @syssi , I will ask :) . At what stage is the work with BLE? I have always used RS485 connection, however, a bank of 3 storages was created and BLE would already be useful to monitor all of them :( |
The implementation has pretty low priority because the demand is very small. Do you have a working setup already and would you offer some testing? |
Thanks for the reply. Yes I have everything and can offer tests :). I even already installed the code esp32-ble-example.yaml. |
2613155
to
19c8d17
Compare
Hm, I tried the example code and this is what I see in the log:
The update interval warning shows up frequently then. I have set it to 30s already... |
I have 2 banks. I am looking forward to being able to monitor the master |
For those of you interested - there is now a Home Assistant integration that allows reading out individual packs and cell voltages over Bluetooth: https://github.com/patman15/BMS_BLE-HA |
68ccac0
to
3977597
Compare
ddc75b9
to
7b1c3fa
Compare
TODOs:
Fixes: #65