-
-
Notifications
You must be signed in to change notification settings - Fork 126
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
Treadmill-modbus #2636
Conversation
…myos-zwift into treadmill-gpio
…-zwift into treadmill-gpio
thanks my idea is to create one with these as settings. is it working? |
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. |
now that i'm thinking of this i didn't finish to if you have a windows pc you can try auto inclination right now going to bed now, we will discuss about this tomorrow |
sounds good! I'll try on windows and we can figure out the Linux tmrw. |
you can use the windows build that the github actions is building right now
Roberto Viola
Software engineer and open source enthusiast
http://robertoviola.cloud
Il giorno sab 5 ott 2024 alle 22:26 jjbrough ***@***.***> ha
scritto:
… sounds good! I'll try on windows and we can figure out the Linux tmrw.
—
Reply to this email directly, view it on GitHub
<#2636 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAALYWA5VBT7X5QV25HDWZLZ2BDRDAVCNFSM6AAAAABPNT6JASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJVGE4DEOBZGA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Hey, just want to follow up.
Dosent look like the build worked. I can try to compile again, but is there
an issue with the source?
…On Sat, Oct 5, 2024, 16:28 Roberto Viola ***@***.***> wrote:
you can use the windows build that the github actions is building right
now
Roberto Viola
Software engineer and open source enthusiast
http://robertoviola.cloud
Il giorno sab 5 ott 2024 alle 22:26 jjbrough ***@***.***> ha
scritto:
> sounds good! I'll try on windows and we can figure out the Linux tmrw.
>
> —
> Reply to this email directly, view it on GitHub
> <
#2636 (comment)>,
> or unsubscribe
> <
https://github.com/notifications/unsubscribe-auth/AAALYWA5VBT7X5QV25HDWZLZ2BDRDAVCNFSM6AAAAABPNT6JASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJVGE4DEOBZGA>
> .
> You are receiving this because you commented.Message ID:
> ***@***.***>
>
—
Reply to this email directly, view it on GitHub
<#2636 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BKEI7URA3XIAS6GUU3WTPTLZ2BDVLAVCNFSM6AAAAABPNT6JASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJVGE4DGNZXGY>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Share a debug log of the issue otherwise I can't see the issue
https://github.com/cagnulein/qdomyos-zwift/wiki/How-do-i-get-the-debug-log-in-case-something-doesn't-work%3F
Roberto Viola
Software engineer and open source enthusiast
http://robertoviola.cloud
Il giorno mer 16 ott 2024 alle 22:56 jjbrough ***@***.***> ha
scritto:
… Hey, just want to follow up.
Dosent look like the build worked. I can try to compile again, but is
there
an issue with the source?
On Sat, Oct 5, 2024, 16:28 Roberto Viola ***@***.***> wrote:
> you can use the windows build that the github actions is building right
> now
>
> Roberto Viola
> Software engineer and open source enthusiast
> http://robertoviola.cloud
>
>
>
> Il giorno sab 5 ott 2024 alle 22:26 jjbrough ***@***.***> ha
> scritto:
>
> > sounds good! I'll try on windows and we can figure out the Linux tmrw.
> >
> > —
> > Reply to this email directly, view it on GitHub
> > <
>
#2636 (comment)>,
>
> > or unsubscribe
> > <
>
https://github.com/notifications/unsubscribe-auth/AAALYWA5VBT7X5QV25HDWZLZ2BDRDAVCNFSM6AAAAABPNT6JASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJVGE4DEOBZGA>
>
> > .
> > You are receiving this because you commented.Message ID:
> > ***@***.***>
> >
>
> —
> Reply to this email directly, view it on GitHub
> <
#2636 (comment)>,
> or unsubscribe
> <
https://github.com/notifications/unsubscribe-auth/BKEI7URA3XIAS6GUU3WTPTLZ2BDVLAVCNFSM6AAAAABPNT6JASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJVGE4DGNZXGY>
> .
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
—
Reply to this email directly, view it on GitHub
<#2636 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAALYWE4N5JFEO5SOD657ODZ33HHTAVCNFSM6AAAAABPNT6JASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJXHE2DONBSGY>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
It's the issue with this not compiling, so I can't access the files. |
ok which is the build error? |
For the Windows build without Python: Mode LastWriteTime Length Name d---- 10/5/2024 8:24 PM output 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
|
No I mean aren't you building on raspberry?
Roberto Viola
Software engineer and open source enthusiast
http://robertoviola.cloud
Il giorno gio 17 ott 2024 alle 23:48 jjbrough ***@***.***> ha
scritto:
… 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
—
Reply to this email directly, view it on GitHub
<#2636 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAALYWGUDTAK5VA7ZRYM2ADZ4AWEHAVCNFSM6AAAAABPNT6JASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRQGY3DEMRWGQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
I was going to, but I think you said above that the linux zwift api build wasn't complete yet. |
Yeah that's right, in the meantime do you have a windows machine?
Roberto Viola
Software engineer and open source enthusiast
http://robertoviola.cloud
Il giorno ven 18 ott 2024 alle 01:34 jjbrough ***@***.***> ha
scritto:
… I was going to, but I think you said above that the linux zwift api build
wasn't complete yet.
—
Reply to this email directly, view it on GitHub
<#2636 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAALYWCRTDUVIM3OTJZZTO3Z4BCRNAVCNFSM6AAAAABPNT6JASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRQHA2DIMJSGI>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
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. |
@jjbrough check my comments in the review, you put some typos in the code |
Reverted changes, now there is an issue resolving |
@jjbrough On the gpiotreadmill.h you have the constant with = = Remove a = |
This reverts commit bf9ee3d.
Linux with increased time