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

Add seplos stacked batteries attributes #153

Open
4 tasks done
ouinouin opened this issue Jan 24, 2025 · 3 comments · May be fixed by #166
Open
4 tasks done

Add seplos stacked batteries attributes #153

ouinouin opened this issue Jan 24, 2025 · 3 comments · May be fixed by #166
Assignees
Labels
enhancement New feature or request

Comments

@ouinouin
Copy link

Checklist

  • I have filled out the template to the best of my ability.
  • I have understood that this integration is only about battery management systems with Bluetooth Low Energy support.
  • This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
  • This issue is not a duplicate of any previous feature requests.

Please describe the enhancement you would like to have.

Hi, first of all thanks for your implementation of the seplos BLE protocol.
I ows 2 set of 48V LIFEPO4 batteries equipped with seplos 3.0 BMS rev 11.
while stacking multiple seplos bms and connecting the bms with RJ45 cable, they do something clever , only the first battery is reporting through BLE , and its reproting all theinfos from all other racks, so for examples intead of having 16 attributes for the cells voltage, i have 32 attributes for the cell voltage hidden behind the deltaV sensor.
but on the official application, we can also see , the voltage for both units , the SOC for bith units etc etc.
i see this is working for the temperatures and for the cells voltages (i see 8 temperatures (4 per rack). it would be nice to see at least as attributes all individual values, specially for voltage and SOC.

my set of 2 racks is available to test if you re interrested, i m not a coder but can fiddle around if necessary

Additional context

No response

@ouinouin ouinouin added the enhancement New feature or request label Jan 24, 2025
@patman15
Copy link
Owner

Hi! I think I can add the data per pack as attributes quite easily. Will give it a try and add a branch for you to test. Ok for you? Just need a calm evening.

@ouinouin
Copy link
Author

Hi, @patman15
happy to give a try , the calm evening is also a prerequisite here :-) .

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

@ouinouin pushed an update to the linked branch, can you check and provide me a debug log? Thanks!

@patman15 patman15 linked a pull request Feb 2, 2025 that will close this issue
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.

2 participants