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

Fix initial move of servo motor #31

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

Conversation

sapics
Copy link
Contributor

@sapics sapics commented Oct 11, 2019

This PR would fix the problem which is reported in #14 .
I guess that it makes same initial move as Servo library
and this could also avoid initial movement with .attach(...,0)
which would be new feature.
And it would also close #25 #26 .

This PR changes the initial state of servo.ticks and
I have not tested this PR yet, because I don't have servo motor, sorry.
Thus, it might be better to apply this PR after someone's test report.
I create this PR with reference to https://stackoverflow.com/questions/40892922/.

And I am not good at English, thus, if you know better English words,
please fix variable name and comments.

We can control initial move of servo motor with initialPulseWidth.
@sapics
Copy link
Contributor Author

sapics commented Oct 11, 2019

This PR add the new method .attach(pin, min, max, initialPulseWidth).
If we could not add new method for compatiblity with Servo library, I will create a new PR which is compatible with Servo library.
But it moves to DEFAULT_PULSE_WIDTH with .attach.

@Dakkaron
Copy link

I know the repo is officially unmaintained, but would it still be possible to merge this?

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.

2 participants