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

No install documentation, unable to add as integration #2

Open
Battlewind opened this issue Oct 30, 2023 · 4 comments
Open

No install documentation, unable to add as integration #2

Battlewind opened this issue Oct 30, 2023 · 4 comments

Comments

@Battlewind
Copy link

I don't know how to properly state that I cannot for the life of me understand how this is added and works. I have two BMS attached via USB and have downloaded the repository via HACS and nothing stands out as to how I can add the sensor to the energy dashboard or anywhere for that matter. Can you provide a version number of HA this worked with and if this will work with the current version? Maybe a small step by step of what to look for?

@ElD4n1
Copy link
Owner

ElD4n1 commented Oct 30, 2023

Hi, sorry i didn't test this with a new HA version for a long time. Please try if the new version v1.0.2 works for you.

@Battlewind
Copy link
Author

I removed it and reinstalled everything. Still getting unknown. I do see this error in the logs. Could it be the naming convention of the BMS?

bmstools: Not adding entity with invalid device info: Invalid device info {'default_name': 'Battery Management System', 'hw_version': '2021.6.25', 'identifiers': {('bmstools', 1)}, 'manufacturer': 'JBD', 'model': 'JBD-SP10S009-L8S-100A-B-K 1ST', 'sw_version': 22} for 'bmstools' config entry: device info needs to either describe a device, link to existing device or provide extra information.

@Battlewind
Copy link
Author

Battlewind commented Oct 31, 2023

Also, there are two BMS connected to this system. Unsure if that makes any difference as well. I did disconnect the one above and tried the second alone and got the same style error for the second one alone. For some reason HA does not want to make this a device properly.

@ElD4n1
Copy link
Owner

ElD4n1 commented Dec 4, 2024

This should be fixed now: https://github.com/ElD4n1/home-assistant-bms-tools-integration/releases/tag/v1.1.0-beta.2

Also, there are two BMS connected to this system. Unsure if that makes any difference as well. I did disconnect the one above and tried the second alone and got the same style error for the second one alone. For some reason HA does not want to make this a device properly.

For multiple devices it is mandatory that you allow the integration to write a unique serial number to the BMS in the last step.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants