cagnulein / qdomyos-zwift

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

[BUG] QZ calculates calories too low inaccurately. #2537

Closed saeonnuri closed 3 weeks ago

saeonnuri commented 3 weeks ago

Describe the bug A clear and concise description of what the bug is. I found this bug when syncing the fit data to Strava for recording into Samsung Health. However, sent fit data all seems fine except too low calories.

When I saw the saved fit file, it saved calories with kCal unit, but it also seems wrong because the 50 min ride would not burn 7000 calories. QZ Fit data.zip

I attached 50 min ride, short test ride to create log, debug log, and profile data in the above data zip. The gears I'm using;

  1. Old indoor bike without any data sharing functions or modules.
  2. M2Me VRFit cadence & power sensor which is used as power sensor in the QZ app. (The sensor can check power value too.)
  3. Samsung watch with QZ watch app to get heart rate.
  4. Meta quest 3 VZFit app connected To Reproduce Steps to reproduce the behavior:

Just ride a few minutes with sensor connected.

Expected behavior A clear and concise description of what you expected to happen.

Calories must be calculated based on power value and time or heart rate.

Screenshots If applicable, add screenshots to help explain your problem. strava wrong calories

Desktop (please complete the following information): Not used to this bug.

Smartphone (please complete the following information):

Append a debug log Check log file inside QZ Fit data.zip

Additional context Check the 50 min ride fit data(This session does not create debug log) and qz profile data in the zip file.

cagnulein commented 3 weeks ago

got the issue, i will try to fix it today

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

Il giorno dom 25 ago 2024 alle 08:33 saeonnuri @.***> ha scritto:

Assigned #2537 https://github.com/cagnulein/qdomyos-zwift/issues/2537 to @cagnulein https://github.com/cagnulein.

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

cagnulein commented 3 weeks ago

done @saeonnuri https://github.com/cagnulein/qdomyos-zwift/actions/runs/10545260553 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

saeonnuri commented 3 weeks ago

Hi, Thank you for your supporting.

Calories calculation now seems fine, however now the heart rate from Samsung Watch(Galaxy watch 7) is disabled. Is it normal in the beta version that you've uploaded here?

My QZ WearOS version in Samsung Watch is 1.1.11

cagnulein commented 3 weeks ago

Yes it's because it works only with the official play store app. I will push it tomorrow and Google will review it in a few days

Join the beta here in the meantime https://play.google.com/apps/testing/org.cagnulen.qdomyoszwift

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

Il giorno dom 25 ago 2024 alle 17:19 saeonnuri @.***> ha scritto:

Hi, Thank you for your supporting.

Calories calculation now seems fine, however now the heart rate from Samsung Watch(Galaxy watch 7) is disabled. Is it normal in the beta version that you've uploaded here?

My QZ WearOS version in Samsung Watch is 1.1.11

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

saeonnuri commented 3 weeks ago

@cagnulein Thank you, I checked all seems fine in the app from Google beta link. I think it's good to close this issue thanks again.

cagnulein commented 3 weeks ago

mmm i didn't push the beta yet, anyway ok the issue is fixed, you will see the update in the beta channel this week hopefully

saeonnuri commented 3 weeks ago

Well, that's curious but it's fixed anyway. Maybe somethings were tangled in my mobile phone about functions related calories. Reinstall and reconfiguring might fix this problem. I'll let you know if it happens again. Thank you for caring this problem.