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

Incorrect constant names or values defined for time conversion #75

Closed
jo3w4rd opened this issue Mar 15, 2016 · 1 comment
Closed

Incorrect constant names or values defined for time conversion #75

jo3w4rd opened this issue Mar 15, 2016 · 1 comment
Labels

Comments

@jo3w4rd
Copy link

jo3w4rd commented Mar 15, 2016

LeapHandController defines constants for time conversion:

const float     NS_TO_S = 1e-6f
const float     S_TO_NS = 1e6f

Be that as it may, a nanosecond is 1e-9 seconds not 1e-6 seconds. Either the constant name is wrong or the value is wrong.

@jo3w4rd jo3w4rd added the bug label Mar 15, 2016
@jo3w4rd
Copy link
Author

jo3w4rd commented Mar 15, 2016

Note, the constant is USED to convert microseconds to seconds, so it is the name, not the value that is wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants