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] QZ Doesn't send send metrics to Peloton app when power sensor is selected in Accessories #2312

Open RickNY opened 2 weeks ago

RickNY commented 2 weeks ago

Describe the bug Normally, the Peloton app will utilize the virtual QZ device to get treadmill metrics such as speed, incline, distance, and HR if you a) use "Connect Devices" in the Peloton app and choose "Treadmill", and then select the "QZ" device listed. Then, if the Peloton device is in a portrait orientation (Peloton limitation), it will display the treadmill's current speed, incline, distance, and HR on corresponding sections of the Peloton app. However, if you also enable a "Power Sensor" in the Accessories and manually select a "Venu 3" Garmin smartwatch that is configured to transmit sensors during a treadmill workout, QZ no longer provides the speed, incline, distance metrics to the Peloton app - but still continues to relay the HR from the QZ virtual device.

To Reproduce (NORMAL behavior): Steps to reproduce the behavior:

  1. Ensure virtual device and virtual bluetooth are enabled. Also, Heart Belt name is manually set to "Venu 3" in QZ
  2. Allow QZ to establish connection to Horizon 7.4 tread and start a Peloton walk or run workout. Use "Connect Device" in Peloton to select "QZ" as treadmill device.
  3. Press Start on QZ once Peloton intro timer finishes
  4. Treadmill starts, and Peloton app on iPad displays treadmill's current speed, incline, distance, and HR. QZ app displays HR tile as it is updated from watch. Cadence tile remains at 0.

To Reproduce (bug behavior):

  1. Ensure virtual device and virtual bluetooth are enabled in QZ
  2. On Venu 3 watch, ensure Treadmill activity is set to transmit sensors and go to the treadmill activity on the watch
  3. in QZ, go to Power Sensor and choose "Refresh Device List", select "Venu 3" and click OK
  4. Save settings and exit/restart QZ
  5. Allow QZ to establish connection to Horizon 7.4 tread and start a Peloton walk or run workout. Use "Connect Device" in Peloton to select "QZ" as treadmill device.
  6. Press Start on QZ once Peloton intro timer finishes
  7. Treadmill starts, and Peloton app on iPad displays 0.0 for treadmill speed, 0.0 for current incline, 0.00 for distance, and the current HR as reported on the watch. Only HR updates. In QZ, both the HR and Cadence tile update.

Expected behavior Enabling the power sensor should still allow QZ to continue transmitting proper FTMS metrics to Peloton app

Screenshots Attached

Desktop (please complete the following information):

Smartphone (please complete the following information):

qz_pel_powerdisabled qz_pel_powerenabled Power_Disabled_debug-Mon_Apr_29_08_09_01_2024.log.txt Power_Enabled_debug-Mon_Apr_29_08_05_56_2024.log.txt

cagnulein commented 2 weeks ago

Fixed! @RickNY stupid mistake! I will send you a new one later today/tomorrow

RickNY commented 2 weeks ago

Great – did that have anything to do with Michelle’s issue as well?

From: Roberto Viola @.> Sent: Monday, April 29, 2024 10:50 AM To: cagnulein/qdomyos-zwift @.> Cc: RickNY @.>; Mention @.> Subject: Re: [cagnulein/qdomyos-zwift] [BUG] QZ Doesn't send send metrics to Peloton app when power sensor is selected in Accessories (Issue #2312)

Fixed! @RickNY https://github.com/RickNY stupid mistake! I will send you a new one later today/tomorrow

— Reply to this email directly, view it on GitHub https://github.com/cagnulein/qdomyos-zwift/issues/2312#issuecomment-2082950021 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFLTVLENA2GNGV7F46ZRMTY7ZM2JAVCNFSM6AAAAABG6LPVMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSHE2TAMBSGE . You are receiving this because you were mentioned. https://github.com/notifications/beacon/AAFLTVNWYJQFUFCYPYWFRFTY7ZM2JA5CNFSM6AAAAABG6LPVMCWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTT4E5FYK.gif Message ID: @. @.> >

cagnulein commented 2 weeks ago

nope, her issue is about android peloton version

Roberto Viola Software engineer and open source enthusiast http://robertoviola.cloud

Il giorno lun 29 apr 2024 alle 17:00 RickNY @.***> ha scritto:

Great – did that have anything to do with Michelle’s issue as well?

From: Roberto Viola @.> Sent: Monday, April 29, 2024 10:50 AM To: cagnulein/qdomyos-zwift @.> Cc: RickNY @.>; Mention @.> Subject: Re: [cagnulein/qdomyos-zwift] [BUG] QZ Doesn't send send metrics to Peloton app when power sensor is selected in Accessories (Issue #2312)

Fixed! @RickNY https://github.com/RickNY stupid mistake! I will send you a new one later today/tomorrow

— Reply to this email directly, view it on GitHub < https://github.com/cagnulein/qdomyos-zwift/issues/2312#issuecomment-2082950021> , or unsubscribe < https://github.com/notifications/unsubscribe-auth/AAFLTVLENA2GNGV7F46ZRMTY7ZM2JAVCNFSM6AAAAABG6LPVMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSHE2TAMBSGE> . You are receiving this because you were mentioned. < https://github.com/notifications/beacon/AAFLTVNWYJQFUFCYPYWFRFTY7ZM2JA5CNFSM6AAAAABG6LPVMCWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTT4E5FYK.gif> Message ID: @. @.> >

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

cagnulein commented 2 weeks ago

sent! let me know!

Roberto Viola Software engineer and open source enthusiast http://robertoviola.cloud

Il giorno lun 29 apr 2024 alle ore 17:01 cagnulein @.***> ha scritto:

nope, her issue is about android peloton version

Roberto Viola Software engineer and open source enthusiast http://robertoviola.cloud

Il giorno lun 29 apr 2024 alle 17:00 RickNY @.***> ha scritto:

Great – did that have anything to do with Michelle’s issue as well?

From: Roberto Viola @.> Sent: Monday, April 29, 2024 10:50 AM To: cagnulein/qdomyos-zwift @.> Cc: RickNY @.>; Mention @.> Subject: Re: [cagnulein/qdomyos-zwift] [BUG] QZ Doesn't send send metrics to Peloton app when power sensor is selected in Accessories (Issue #2312)

Fixed! @RickNY https://github.com/RickNY stupid mistake! I will send you a new one later today/tomorrow

— Reply to this email directly, view it on GitHub < https://github.com/cagnulein/qdomyos-zwift/issues/2312#issuecomment-2082950021> , or unsubscribe < https://github.com/notifications/unsubscribe-auth/AAFLTVLENA2GNGV7F46ZRMTY7ZM2JAVCNFSM6AAAAABG6LPVMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSHE2TAMBSGE> . You are receiving this because you were mentioned. < https://github.com/notifications/beacon/AAFLTVNWYJQFUFCYPYWFRFTY7ZM2JA5CNFSM6AAAAABG6LPVMCWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTT4E5FYK.gif> Message ID: @. @.> >

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

RickNY commented 2 weeks ago

Great thanks. Won't be for several hours after I'm off from work. On Apr 29, 2024, at 11:12 AM, Roberto Viola @.***> wrote: sent! let me know!

Roberto Viola

Software engineer and open source enthusiast

http://robertoviola.cloud

Il giorno lun 29 apr 2024 alle ore 17:01 cagnulein @.***> ha

scritto:

nope, her issue is about android peloton version

Roberto Viola

Software engineer and open source enthusiast

http://robertoviola.cloud

Il giorno lun 29 apr 2024 alle 17:00 RickNY @.***> ha

scritto:

Great – did that have anything to do with Michelle’s issue as well?

From: Roberto Viola @.***>

Sent: Monday, April 29, 2024 10:50 AM

To: cagnulein/qdomyos-zwift @.***>

Cc: RickNY @.>; Mention @.>

Subject: Re: [cagnulein/qdomyos-zwift] [BUG] QZ Doesn't send send metrics

to Peloton app when power sensor is selected in Accessories (Issue #2312)

Fixed! @RickNY https://github.com/RickNY stupid mistake! I will send

you a new one later today/tomorrow

Reply to this email directly, view it on GitHub <

https://github.com/cagnulein/qdomyos-zwift/issues/2312#issuecomment-2082950021>

, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AAFLTVLENA2GNGV7F46ZRMTY7ZM2JAVCNFSM6AAAAABG6LPVMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSHE2TAMBSGE>

.

You are receiving this because you were mentioned. <

https://github.com/notifications/beacon/AAFLTVNWYJQFUFCYPYWFRFTY7ZM2JA5CNFSM6AAAAABG6LPVMCWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTT4E5FYK.gif>

Message ID: @. @.> >

Reply to this email directly, view it on GitHub

https://github.com/cagnulein/qdomyos-zwift/issues/2312#issuecomment-2082976199,

or unsubscribe

https://github.com/notifications/unsubscribe-auth/AAALYWELCT2RA6IL6UBOFYTY7ZOCRAVCNFSM6AAAAABG6LPVMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSHE3TMMJZHE

.

You are receiving this because you were assigned.Message ID:

@.***>

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

RickNY commented 2 weeks ago

I'm not sure what speed was being reported in the speed tile in both QZ as well as in Peloton -- but it was not the speed of the treadmill. I think it may have been taking the speed portion of the speed/cadence broadcast? The speed was all over the place, so I suspect that was the speed that was being reported. Attaching debug log as well as screenshot again of the NRF capture from the watch.
debug-Mon_Apr_29_17_52_17_2024.log.txt garmin_spd_cadence

cagnulein commented 2 weeks ago

my fault, sorry, it was about this #2121 #2141 fixed, i'm send you another one

RickNY commented 2 weeks ago

OK, just tried the new one for a quick 5 minute walk, and it seemed to work properly in all regards this time!

cagnulein commented 2 weeks ago

that's great! keep me posted in case of issues!

stale[bot] commented 1 day 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.