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

Eliminate double rounding-errors #4

Open
aquila12 opened this issue Dec 11, 2015 · 2 comments
Open

Eliminate double rounding-errors #4

aquila12 opened this issue Dec 11, 2015 · 2 comments

Comments

@aquila12
Copy link
Contributor

Currently, output from the IK is in radians (as a float). This is converted into degrees (as an integer), and later into microseconds (also as an integer). The second conversion is done within the servo library itself, which also exposes a writeMicroseconds method.

Some rounding error could be eliminated by directly converting from radians into microseconds, and omitting the second conversion (and the intermediate, granular, degree measurement).

@lexbailey
Copy link
Member

Would switching to this require a recalibration for a given working arm?

@aquila12
Copy link
Contributor Author

No; well, sort of - for a given set of existing cal values, we could
convert them (and provide helpers for that).

It might be worth creating an example sketch for calibration - disabling
all servos to start with, and setting each in turn to set it up properly.

On 11 December 2015 at 18:39, Daniel Bailey [email protected]
wrote:

Would switching to this require a recalibration for a given working arm?


Reply to this email directly or view it on GitHub
#4 (comment).

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