-
-
Notifications
You must be signed in to change notification settings - Fork 14
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 Ective LiFePo4 batteries to your integration #82
Comments
Hi Andreas! Shouldn't be too hard. Are you using this app: https://play.google.com/store/apps/details?id=com.topband.smartpower.ective ? Because there are multiple and the store.
Need to do a release and I'm a bit busy this week, but will have a look! |
Hi Patrick. I want to understand to add a new type of battery in your integration. I think there are many battery brands that use this type of BMS. BR |
If you want to try yourself, have a look at the contribution guidelines for details. |
@Karatzie Hi Andreas, |
@patman15 Hi Patrick. I have to try to analyze your code. I have analyzed the last missing values of the communication log. Please add this type of battery in your code. BR |
No worries, with your great prework it shouldn't be too difficult. |
@Karatzie Hi Andreas! I'm almost done, but I need info on how to identify the battery. Can you send me a screenshot of the advertisement of the battery from nRF connect? Or at least whether the batteries have a name schema? From the screenshots of the app, I would guess "NWJ20*" would be a good name identifier, correct? |
Hi Patrick. this is from the log attached to the first post. `nRF Connect, 2024-11-11
|
I am not the owner of the battery. |
No that I have seen, what I mean is this information: #59 (comment) |
we have to wait for the evening. |
My friend established a telegram group with 98 members at the moment. |
Hi Andreas, that is nice of you, but it sounds a bit overwhelming for me. Working on the integration already consumes a lot of my spare time. 😉 Happy to assist if someone needs support, though. |
The Ective battery was not found by the bms-ble-ha. |
Assumed so after seeing your picture, please try the updated branch. |
I am not able to send you the log directly. |
This won't help, I do not recommend changing it! I sent you a PM from my HA account so I hope you are able to reply to it. |
@Karatzie I pushed an update, I think this will fix the issue (hopefully). |
Hi Patrick. I was not at home over night. You will find an error log as attachment. We will che your new update. BR Andreas |
That was just a duplicate log message. I fixed that together with another patch to make sure it works. |
Hi Patrick, here are the Logs of the last update. home-assistant_bms_ble_2024-11-28T06-19-38.629Z.log home-assistant_bms_ble_2024-11-28T19-12-29.291Z.log Please take a look into the logs. BR Andreas |
Hi Andreas! Does your message mean it works and you are happy? I can see still some errors in the log, but all of them are before 2024-11-28 20:11:19, after that point in time, everything is fine. |
Hi Patrick. Thank you for your support for the Ective BMS. BR |
Thanks for the feedback, I'll add it to the next release. |
Checklist
Is your feature request related to a problem? Please describe.
I would like to add a new battery in your integration
Describe the solution you'd like
You will find my documents below.
Additional context
Hi Patrick,
The Ective LiFePo4 batteries have also a signal exchange.
I have not a signal list with the description of the byte order but I have reverse enginered the signals.
I have a nRF connect list and know 95% of the signals.
Do you need additional information for an integration?
This is the log-file:
Log 2024-11-11 12_50_52.txt
This are my investigations.
Daten.xlsx
Please check the documents and let me know if it possible to add the Ective battery.
Thanks
Andreas (Karatzie)
The text was updated successfully, but these errors were encountered: