Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] JTX Fitness Sprint 5 disconnects when changing incline #1762

Closed
kangaroomadman opened this issue Nov 1, 2023 · 14 comments
Closed

[BUG] JTX Fitness Sprint 5 disconnects when changing incline #1762

kangaroomadman opened this issue Nov 1, 2023 · 14 comments
Assignees
Labels
bug Something isn't working wontfix This will not be worked on

Comments

@kangaroomadman
Copy link

Describe the bug
When using with the JTX Fitness Sprint 5 Treadmill (using iConsole+), any change in incline will cause the treadmill to start altering the incline but then disconnect from bluetooth.

The 'JTX Fitness Sprint Treadmill' compatibility setting was used in QZ.

To Reproduce
Steps to reproduce the behavior:

  1. Turn on treadmill
  2. Open QZ and wait for it to connect
  3. Start the treadmill
  4. Attempt to increase incline

Expected behavior
Incline to change and bluetooth to remain for metrics collection.

Tablet (please complete the following information):

  • Device: Android Fire HD 8 (12th Generation)
  • OS: Fire OS
  • Version 8.3.2.0

Append a debug log

debug-Mon_Oct_30_21_02_28_2023.log

Additional context

A capture of Kinomap successfully controlling the incline can be found here:
btsnoop_hci.log

Each time this successfully controlled the incline, the same command as QZ was sent, but the different being that it was repeated 1 second later.

f0ac01d301646439 sent at 41.38
f0ac01d301646439 send at 42.44
f0ac01d30264643a sent at 55.71
f0ac01d30264643a sent at 56.73

At the end of the capture, the incline was increased by several points which caused multiple commands to be sent by Kinomap without repeats. This caused a bluetooth disconnect.

f0ac01d30564643d sent at 77.16 (there was no f0bc... response for this)
f0ac01d30764643f sent at 78.19
f0ac01d309646441 sent at 79.21
f0ac01d30b646443 sent at 80.24
After the f0bc... response for the command above, the connection drops.

@kangaroomadman kangaroomadman added the bug Something isn't working label Nov 1, 2023
@cagnulein
Copy link
Owner

#598

Thanks I will check it asap

@cagnulein
Copy link
Owner

@kangaroomadman do you have also a PC? we can test quickly my solution on the PC build. Let me know

@kangaroomadman
Copy link
Author

@kangaroomadman do you have also a PC? we can test quickly my solution on the PC build. Let me know

Yep, Windows 11 do?

@cagnulein
Copy link
Owner

yep! let me create the branch for you!

@cagnulein
Copy link
Owner

ok @kangaroomadman did it. It's building now https://github.com/cagnulein/qdomyos-zwift/actions/runs/6730276253
When the build it's completed you will see the files of the windows version at the bottom of the page.
What I did in this version is:

  • i added the second inclination rep. as kinomap does
  • i split inclination requester higher than 1% in several request each one of 0.5%

let's see if it's better

Let me know!

@cagnulein
Copy link
Owner

@cagnulein
Copy link
Owner

Copy link

stale bot commented Nov 17, 2023

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.

@stale stale bot added the wontfix This will not be worked on label Nov 17, 2023
@stale stale bot closed this as completed Nov 25, 2023
@cagnulein
Copy link
Owner

@kangaroomadman any news?

@kangaroomadman
Copy link
Author

@cagnulein Sorry, I did test the changes but they made no difference. Looking back through the logs and the packets sniffed from using Kinomap, I can see the response to incline requests is different. Perhaps something needs to be set up differently.

I plan to see the initialisation differences between this and Kinomap to try and see if I can get it to work. Limited time at the moment but I’ll get some things tested when I can.

@cagnulein
Copy link
Owner

cagnulein commented Nov 30, 2023 via email

@cagnulein
Copy link
Owner

@kangaroomadman did you test this #2223 (comment) ?

@kangaroomadman
Copy link
Author

@kangaroomadman did you test this #2223 (comment) ?

@cagnulein Sorry, no - My treadmill has been inaccessible for a while so not had a chance. Has there been improvement under #2223?

@cagnulein
Copy link
Owner

@kangaroomadman i guess so, ask it to the other user ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working wontfix This will not be worked on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants