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

Treadmill-modbus #2636

Closed
wants to merge 147 commits into from
Closed

Treadmill-modbus #2636

wants to merge 147 commits into from

Conversation

jjbrough
Copy link

@jjbrough jjbrough commented Oct 5, 2024

Linux with increased time

cagnulein and others added 30 commits January 5, 2022 23:52
@cagnulein
Copy link
Owner

thanks my idea is to create one with these as settings. is it working?

@jjbrough
Copy link
Author

jjbrough commented Oct 5, 2024

I'm trying! I changed poolDeviceTime = 400 from 200. Is that the time that the switch is "on?" wasn't sure if it was that one or the GPIO_KEEP_MS = 50.

@cagnulein
Copy link
Owner

now that i'm thinking of this i didn't finish to
implements the zwift api on linux.
#1934
anyway let's build and check that you can change inclination from qz tile

if you have a windows pc you can try auto inclination right now

going to bed now, we will discuss about this tomorrow

@jjbrough
Copy link
Author

jjbrough commented Oct 5, 2024

sounds good! I'll try on windows and we can figure out the Linux tmrw.

@cagnulein
Copy link
Owner

cagnulein commented Oct 5, 2024 via email

@jjbrough
Copy link
Author

jjbrough commented Oct 16, 2024 via email

@cagnulein
Copy link
Owner

cagnulein commented Oct 16, 2024 via email

@jjbrough
Copy link
Author

It's the issue with this not compiling, so I can't access the files.

@cagnulein
Copy link
Owner

ok which is the build error?

@jjbrough
Copy link
Author

For the Windows build without Python:

Mode LastWriteTime Length Name


d---- 10/5/2024 8:24 PM output
d---- 10/5/2024 8:24 PM appx
Copy-Item: D:\a_temp\217c0781-43ec-48fe-8ee5-c04439ad34ec.ps1:11
Line |
11 | cp qdomyos-zwift.exe output/
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| Cannot find path 'D:\a\qdomyos-zwift\qdomyos-zwift\src\debug\qdomyos-zwift.exe' because it does not exist.
Error: Process completed with exit code 1.

For linux build:

/opt/hostedtoolcache/Python/3.11.10/x64/bin/python3 -m aqt install-qt linux desktop 5.15.2 --outputdir /home/runner/work/qdomyos-zwift/Qt --modules qtnetworkauth qtcharts qtserial
INFO : aqtinstall(aqt) v3.1.18 on Python 3.11.10 [CPython GCC 9.4.0]
WARNING : Specified modules ['qtserial'] did not exist when this version of aqtinstall was released. This may not install properly, but we will try our best.
ERROR : The packages ['qtserial'] were not found while parsing XML of package information!
==============================Suggested follow-up:==============================

  • Please use 'aqt list-qt linux desktop --modules 5.15.2 ' to show modules available.
    Error: Error: The process '/opt/hostedtoolcache/Python/3.11.10/x64/bin/python3' failed with exit code 1

@cagnulein
Copy link
Owner

cagnulein commented Oct 17, 2024 via email

@jjbrough
Copy link
Author

I was going to, but I think you said above that the linux zwift api build wasn't complete yet.

@cagnulein
Copy link
Owner

cagnulein commented Oct 17, 2024 via email

@jjbrough
Copy link
Author

Yes I do! I can run it on windows in the meantime, but it just looks like when trying to compile it I got the error noted above.

@cagnulein
Copy link
Owner

@jjbrough check my comments in the review, you put some typos in the code

@jjbrough jjbrough marked this pull request as draft October 18, 2024 23:45
@jjbrough jjbrough marked this pull request as ready for review October 19, 2024 00:24
This reverts commit 3c1a1e8, reversing
changes made to 3e32e0a.
@jjbrough
Copy link
Author

Reverted changes, now there is an issue resolving

@cagnulein
Copy link
Owner

@jjbrough On the gpiotreadmill.h you have the constant with = =

Remove a =

@jjbrough jjbrough closed this Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants