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

added move and reset to multistepper #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zybkin
Copy link

@zybkin zybkin commented Dec 5, 2017

Added a function to multistepper which moves servos synchronously like moveTo, but in relative coordinates.
Also added reset function to multistepper which set all steppers to zero pos.

@aster94
Copy link

aster94 commented Sep 25, 2019

@waspinator please could you merge?
Also if you are willing to accept PRs I will bring to multistepper other functions from Accelstepper

@waspinator
Copy link
Owner

@aster94 Thanks for your interest in helping out the project. The focus of this repo is to provide an easy way for Arduino IDE users to access the library. It's best to discuss feature additions in http://groups.google.com/group/accelstepper, and once they get included I will merge them here.

If you are unsuccessful in getting your changes into the original source I can consider creating a branch and adding them here.

@aster94
Copy link

aster94 commented Sep 25, 2019

Hi @waspinator thanks for the fast answer, I will write in the google group proposing my changes

@aster94
Copy link

aster94 commented Sep 26, 2019

@zybkin the original author is willing to implement the changes, can you confirm me that the function move() is working smootly?

@zybkin
Copy link
Author

zybkin commented Sep 29, 2019

@zybkin the original author is willing to implement the changes, can you confirm me that the function move() is working smootly?

This was long ago, but afair it worked well and smooth enough.

@aster94
Copy link

aster94 commented Sep 30, 2019

Thanks, I hope that soon it will be inside the library

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