smagicld / xiaoxiangBMS

iOS app for Xiaoxang BMS
84 stars 9 forks source link

Regression: v1.3.0 no longer shows cell voltages and BMS Infos #226

Closed cnrd closed 1 year ago

cnrd commented 1 year ago

Hi

After updating to v1.3.0 the app no longer shows cell voltages and there is also information missing in the BMS Infos field 1E0DC5E2-F611-407E-93B7-D2C422FA7DE4 and will now ask for a password when entering config, which it did not before EFC932C4-20DB-4652-95D5-B1D2C5236C85

I can still control the BMS relay, to turn on/off the power.

The BMSTool app still shows the cell voltages 814A1054-2E9C-42F2-99F1-41F32412702D

and also correctly shows the BMS Infos 900B183D-60CA-4F00-9C0A-041391D473FE

Here comes the weird part, if I open the BMSTool app and from that open the XiaoxiangBMS through the cogwheel, then the XiaoxiangBMS app seems to behave correctly 1D817253-F04A-412E-A2CD-EAAEFE6FDEEA

However, voltage calibration does not seem to show up, even though I did calibrate the BMS before the app update (not sure if this is expected behavior?) DC6E926C-FA7B-4C6F-948B-81169FCD3E50

smagicld commented 1 year ago

This is very wired / was not able to reproduce it on any of our test BMS.

Have you tried to kill all the Apps and start Xiaoxiang BMS only?

Please also provide me the logs, maybe I can find something there:

  1. Click on About screen -> tap the build number
  2. go back, connect to bms. wait some time
  3. go back to about screen -> tap the build number
  4. send me the email

thanks

cnrd commented 1 year ago

Thanks for the quick reply!

I only installed the BMSTool app when XiaoxiangBMS didn’t work, so I am sure that there was no interference originally.

I will get you the logs asap.

cnrd commented 1 year ago

Just sent you a mail after doing the following steps:

1.: enabled debug 2.: connected to BMS 3.: waited a bit 4.: went to the config menu 5.: canceled out of the pin screen 6.: turned the battery off (which did click the relay) 7.: turned the battery on 8.: disabled debug

smagicld commented 1 year ago

Thanks for the logs. I see in your case only INFO and Device Name is fetched, no CELL Info. I will try to find out where is the problem. will let you know once I've fond something.

smagicld commented 1 year ago

Thanks

I’ve found the Problem. But are you sure it used to work before with this BMS? In my opinion impossible.

Anyhow I will add now support for this BMS Version, will add you as beta user so probably tomorrow you’ll get email from apple (TestFlight) with beta version to test (1.3.1)

Cheers Lukas

On 11 Mar 2023, at 20:12, Conrad Juhl Andersen @.***> wrote:

Just sent you a mail after doing the following steps:

1.: enabled debug 2.: connected to BMS 3.: waited a bit 4.: went to the config menu 5.: canceled out of the pin screen 6.: turned the battery off (which did click the relay) 7.: turned the batter on 8.: disabled debug

— Reply to this email directly, view it on GitHub https://github.com/smagicld/xiaoxiangBMS/issues/226#issuecomment-1464987986, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHW32ZEK76I6OTKW5L73N4DW3TFARANCNFSM6AAAAAAVXTOBNY. You are receiving this because you commented.

cnrd commented 1 year ago

Thanks for the quick fix!

Completely sure, I’ve had the BMS since summer and had access to all of the features with both my current and previous iPhone.

If possible, please use the email address in the body of the email and not the one I sent the debug log from, I unfortunately was not able to sent it from my primary address (which is the one in the body).

smagicld commented 1 year ago

ok. the 1.3.1 (build: 7662) is submitted to apple. should be released within 1 - 2 days.

cnrd commented 1 year ago

Just got the build, working now, thanks!

DF55D291-0684-412A-8CF4-A6E3AFE43D81

smagicld commented 1 year ago

Cool. Happy we could solve it. And if you have some time - would be great you could leave AppStore rating. Thanks!