[BUG] Kingsmith Walkingpad P1 no cadence reported

[hubaiz]

No cadence is reported by Walkingpad P1

To Reproduce
Steps to reproduce the behavior:
1.Connect to walkingpad p1 with BT
2. start
3. the cadence field is 0 and does not change

Expected behavior
A cadence should be reported

Screenshots

Smartphone (please complete the following information):

• Device: xiaomi MI 9 Lite
• OS: Android 10 (MIUI 12.0.10

Append a debug log
debug-Mon_Apr_22_19_37_39_2024.log

Additional context
in the ks fit application the steps are reported

[cagnulein]

Thanks! I will check it! One question : do you see the cadence in the original walking pad app? Roberto Viola Software engineer and open source enthusiast http://robertoviola.cloud Il giorno lun 22 apr 2024 alle 19:54 hubaiz ***@***.***> ha scritto:

…

Assigned #2304 <#2304> to @cagnulein <https://github.com/cagnulein>. — Reply to this email directly, view it on GitHub <#2304 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAALYWANJIVH3MJ25HGPNMDY6VFEXAVCNFSM6AAAAABGTHX36GVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGU3DGNZRGA4TINQ> . You are receiving this because you were assigned.Message ID: ***@***.***>

[hubaiz]

Yes, i can see it.

[cagnulein]

Steps as an incremental right? So I guess I can add it too! I will let you know in the next days!

[hubaiz]

Yes, and Thanks :)

[cagnulein]

Note for me: byte 7

[cagnulein]

@hubaiz done!
https://github.com/cagnulein/qdomyos-zwift/actions/runs/8797095704

Google is very slow in these days, reviewing the app on the store, even the beta ones. For this reason, in order to check right now if the patch that I did is right, you can use the android build in the link (check at the bottom of the page in about 1 hour). In order to test it you need first to remove the current QZ version on your phone, install the one that you downloaded.
This version is a 15 minutes trial (you can restart it as many times as you want) and I can unlock it if you want, send me an email to roberto.viola83@gmail.com mentioning this ticket and that you are on android.
Let me know if you have any questions

[hubaiz]

New debug log:
debug-Tue_Apr_23_11_36_30_2024.log

[cagnulein]

good news step counts works (you can see enabling the step count tile). let me fix the cadence Roberto Viola Software engineer and open source enthusiast http://robertoviola.cloud Il giorno mar 23 apr 2024 alle 11:41 hubaiz ***@***.***> ha scritto:

…

New debug log: debug-Tue_Apr_23_11_36_30_2024.log <https://github.com/cagnulein/qdomyos-zwift/files/15074747/debug-Tue_Apr_23_11_36_30_2024.log> — Reply to this email directly, view it on GitHub <#2304 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAALYWA5ISYZYYOE2O4HZ4TY6YUC5AVCNFSM6AAAAABGTHX36GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZRHA3DIMRYG4> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[hubaiz]

Yes now that you mentioned I enabled the steps tile and see it counting the steps.
My mistake that I assumed steps=cadence.

[cagnulein]

ok i guess i fixed cadence too now https://github.com/cagnulein/qdomyos-zwift/actions/runs/8798362310

[hubaiz]

Did checkout the new build but still no cadence is reported to zwift, also in the UI I only see the steps increasing but no cadence data.
Here is the log:
debug-Tue_Apr_23_13_22_51_2024.log

[cagnulein]

sorry i forgot a casting

https://github.com/cagnulein/qdomyos-zwift/actions/runs/8800894239

[hubaiz]

Thanks, there is now cadence, as in app as in Zwift.
A bit fluctuating but probably it is because of the low speed.
log:
debug-Tue_Apr_23_15_47_39_2024.log

[cagnulein]

i added a 5s average it should be better https://github.com/cagnulein/qdomyos-zwift/actions/runs/8801774094

[hubaiz]

Thanks, unfortunately I was traveling today and the build didn't finish till then. I can test it only on Sunday or Monday.

[cagnulein]

No problem! Let me know!

[hubaiz]

Tested it and now the cadance is working and it is more stable. Works in Zwift to.

[cagnulein]

Glad to hear it! Il giorno dom 28 apr 2024 alle 18:02 hubaiz ***@***.***> ha scritto:

…

Tested it and now the cadance is working and it is more stable. Works in Zwift to. — Reply to this email directly, view it on GitHub <#2304 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAALYWFGCD6P3C4D7PXCQW3Y7UMRPAVCNFSM6AAAAABGTHX36GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBRGUZTGMRYGA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.