cagnulein / qdomyos-zwift

Zwift bridge for smart treadmills and bike/cyclette
https://www.qzfitness.com/
GNU General Public License v3.0
337 stars 103 forks source link

[BUG] Kingsmith Walkingpad P1 no cadence reported #2304

Open hubaiz opened 3 weeks ago

hubaiz commented 3 weeks ago

No cadence is reported by Walkingpad P1

To Reproduce Steps to reproduce the behavior: 1.Connect to walkingpad p1 with BT

  1. start
  2. the cadence field is 0 and does not change

Expected behavior A cadence should be reported

Screenshots Screenshot_2024-04-22-19-47-03-694_org cagnulen qdomyoszwift

Smartphone (please complete the following information):

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 commented 3 weeks ago

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 https://github.com/cagnulein/qdomyos-zwift/issues/2304 to @cagnulein https://github.com/cagnulein.

— Reply to this email directly, view it on GitHub https://github.com/cagnulein/qdomyos-zwift/issues/2304#event-12563710946, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAALYWANJIVH3MJ25HGPNMDY6VFEXAVCNFSM6AAAAABGTHX36GVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGU3DGNZRGA4TINQ . You are receiving this because you were assigned.Message ID: @.***>

hubaiz commented 3 weeks ago

Yes, i can see it. Screenshot_2024-04-22-20-00-47-760_com kingsmith xiaojin

cagnulein commented 3 weeks ago

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

hubaiz commented 3 weeks ago

Yes, and Thanks :)

cagnulein commented 3 weeks ago

Note for me: byte 7

cagnulein commented 3 weeks ago

@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 commented 3 weeks ago

New debug log: debug-Tue_Apr_23_11_36_30_2024.log

cagnulein commented 3 weeks ago

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 https://github.com/cagnulein/qdomyos-zwift/issues/2304#issuecomment-2071864287, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAALYWA5ISYZYYOE2O4HZ4TY6YUC5AVCNFSM6AAAAABGTHX36GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZRHA3DIMRYG4 . You are receiving this because you were mentioned.Message ID: @.***>

hubaiz commented 3 weeks ago

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

cagnulein commented 3 weeks ago

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

hubaiz commented 3 weeks ago

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 commented 3 weeks ago

sorry i forgot a casting

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

hubaiz commented 3 weeks ago

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 commented 3 weeks ago

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

hubaiz commented 3 weeks ago

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

cagnulein commented 3 weeks ago

No problem! Let me know!

hubaiz commented 2 weeks ago

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

cagnulein commented 2 weeks ago

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 https://github.com/cagnulein/qdomyos-zwift/issues/2304#issuecomment-2081533280, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAALYWFGCD6P3C4D7PXCQW3Y7UMRPAVCNFSM6AAAAABGTHX36GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBRGUZTGMRYGA . You are receiving this because you were mentioned.Message ID: @.***>

stale[bot] commented 3 days ago

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.