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

JYQD.enableActive should be LOW instead of HIGH #79

Open
jbrichau opened this issue Jun 22, 2022 · 1 comment
Open

JYQD.enableActive should be LOW instead of HIGH #79

jbrichau opened this issue Jun 22, 2022 · 1 comment

Comments

@jbrichau
Copy link

In the latest of Sunray (1.0.276), the line the correction in amRobotDriver.cpp was reverted again.
I had to redo the change locally to get my mow motor running:

JYQD.enableActive = LOW; // enable active level (LOW or HIGH)

It was changed to the (for me at least) correct value in 23ed0e0
But was changed back to the (for me) incorrect value in f05bf90

@greymfm
Copy link
Member

greymfm commented Jul 24, 2022

For the original JYQD (from the Marotronics shop), it should be HIGH - for the clones (from Amazon etc.) it should be LOW :-)

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

No branches or pull requests

2 participants