cagnulein / qdomyos-zwift

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

Qz sets inclination to -15 on sole f85 treadmill, whenever zwift sends negative inclination #2552

Closed Santa87-cyber closed 2 months ago

Santa87-cyber commented 2 months ago

Describe the bug When using zwift on a hilly route, whenever the route goes downhill I.e. negative inclination, QZ sets inclination to 15 on my sole f85 treadmill.

To Reproduce Steps to reproduce the behavior:

  1. Open QZ and connect to Sole F85
  2. Open Zwift and connect to wahoo tread 003.
  3. Find a route that has negative inclination
  4. See that inclination on treadmill increases to 15, whenever inclination drops below 0 in zwift

Expected behavior I am unsure what I would expect, as treadmill cannot decline but only goes from 0-15. I think I would however expect incline to be 0, whenever zwift incline was <= 0, and 15 whenever zwift incline was >=15.

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

Smartphone (please complete the following information):

Append a debug log

Follow this guide https://github.com/cagnulein/qdomyos-zwift/wiki/How-do-i-get-the-debug-log-in-case-something-doesn't-work%3F

Additional context Add any other context about the problem here.

cagnulein commented 2 months ago

ok thanks, i need the debug log to check it

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

Il giorno gio 29 ago 2024 alle ore 14:52 Santa87-cyber < @.***> ha scritto:

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

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

Santa87-cyber commented 2 months ago

inbound6238579147790698094.zip

cagnulein commented 2 months ago

@Santa87-cyber done! https://github.com/cagnulein/qdomyos-zwift/actions/runs/10615887180

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

Santa87-cyber commented 2 months ago

I am unsure if I have grabbed the right version, or if I should change a setting or something, but the issue seems to persist for me: as soon as inclination drops to -1 in zwift, inclination on treadmill goes to 15.

cagnulein commented 2 months ago

grab another debug log, i did a critical part of code without the possibility to test it so i could did something wrong

Santa87-cyber commented 2 months ago

Alright, will do tomorrow, it's getting quite late here. Thank you for all your help

Santa87-cyber commented 2 months ago

inbound4617813242840690822.zip inbound6894115374025363586 inbound3805644962983787510

cagnulein commented 2 months ago

Good Morning @Santa87-cyber unfortunately the zip is corrupted, can you upload it again? thanks

Santa87-cyber commented 2 months ago

inbound6168197252105030235.zip

Santa87-cyber commented 2 months ago

Try this one. If still corrupted I will have to create a new log 🙃

cagnulein commented 2 months ago

got it now, i'm checking it

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

Il giorno ven 30 ago 2024 alle ore 09:36 Santa87-cyber < @.***> ha scritto:

Try this one. If still corrupted I will have to create a new log 🙃

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

cagnulein commented 2 months ago

ok i guess i have a workaround in the meantime. go into the qz settings, treadmill, and set the min inclination to 0 instead of -100 this should fix the issue. also, in the meantime, could you please download the free app nrfconnect from the store, connect to the treadmill with it and go into the characteristics page, scroll to the ftms service and check for a inclination supported characteristic. could you please share me a screenshot of it? i would like to check why i can't read it. Thanks

Santa87-cyber commented 2 months ago

Tried connecting to treadmill through app. Could not find anything in the app called characteristics or ftms sevice inbound6315463658727423236

cagnulein commented 2 months ago

tes click connect there and then you will see the list of all the services

Santa87-cyber commented 2 months ago

inbound5685527962759967679 inbound3663764882449213962

Is this what you are looking for? Should I Bond the treadmill?

Santa87-cyber commented 2 months ago

inbound3772064082768386805 inbound7106074573486957681 inbound208819695834788782 inbound8171154266206341769

cagnulein commented 2 months ago

thanks! in the meantime try to change the min inclination setting and let me know if it works correctly

Santa87-cyber commented 2 months ago

Just tested with min inclination set to 0, and it seemed to do the trick.. will that also work in the app in Google play, or only in the version you have here on github?

cagnulein commented 2 months ago

yes it should work also on the play store version. i will check if i can do this automatically with the patch so anyone in this situation would not to do this

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

Il giorno ven 30 ago 2024 alle 12:51 Santa87-cyber @.***> ha scritto:

Just tested with min inclination set to 0, and it seemed to do the trick.. will that also work in the app in Google play, or only in the version you have here on github?

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

stale[bot] commented 2 months 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.