search

VincentMasselis / TPMS-advanced

Android app for BLE TPMS made by Sysgration
Apache License 2.0
8 stars 3 forks source link

15 Psi offset #28

Closed 2solt closed 3 months ago

2solt commented 6 months ago

Describe the bug For me there is a 15Psi (atmoshperic) differnce compare to the SYTPMS app. (pecham sensor) Currenly showing: TPMS-Advanced: Front: 47 psi Rear 51 psi SYTPMS: Front: 32.2 Psi Rear: 36.3 Psi

Expected behavior Not shure wich one show the correct value when the tyres are inflated, I have not verified it with another device.

VincentMasselis commented 5 months ago

Hello @2solt and thank you for contributing for the app !

I'll check on my side which values are correct between TPMS-Advanced and the original Pecham app, I will try with my inflator which display the current pressure to check which app is wrong

2solt commented 5 months ago

Thanks for creating it! I just veirfied on the petrol station and it looks like TPMS-Advanced is off by 15 Psi Here is my raw data: 0x0303A5270308425208FF80201401D48344 (wich is 32.2Psi, 18.0°C, 3.2V in SYTPMS and 47Psi in TPMS-Advanced, the temperature is the same 18.0°C)

zenyagami commented 4 months ago

I tried the fix in the pr but interesting enough I have different data, I'll attach the result hex, but I'm the app shows 3.0bar and in the app (using the psi branch) it displays 0.6 bar 🙉 I'm not sure if the bye 3-4 should be used only from here, https://github.com/andi38/TPMS I'll do more test later. Thanks a lot for the effort

2solt commented 4 months ago

Works for me (and its aligned with SYTPMS) atpms

VincentMasselis commented 3 months ago

@2solt Thanks for trying it !

I also have to try on my side, if OK, I'll create a release, probably the next month

VincentMasselis commented 3 months ago

A beta with includes the fix was published this week: https://play.google.com/apps/testing/com.masselis.tpmsadvanced