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

Suggestion #8

Open
moffile opened this issue Jan 15, 2022 · 2 comments
Open

Suggestion #8

moffile opened this issue Jan 15, 2022 · 2 comments

Comments

@moffile
Copy link

moffile commented Jan 15, 2022

Not sure if you're interested in making any changes at this point, but on line 45 of Turntable.ino there is a hard-coded scale factor of "4.4". This hard-coded value seems to be scaling the number of steps-per-revolution based on the gear ratio of the turntable. This is important, especially if someone is trying to use this code on a turntable that is different from the original design, as not modifying this value correctly ruins the follow-on calculations for number of steps per revolution, and it might be hard for some users to debug why they can't get an exact 360deg revolution from the code. I suggest creating a const for this value and adding a comment to it so that users know to modify this value based on the gear ratio of the turntable they use.

@QLRO
Copy link
Owner

QLRO commented Sep 10, 2022

Hi @moffile ,

Sorry for the very late reply. As you can see, I don't use github often.

I do remember I did some kind of calculations, probably using the gear ratio, but I have forgot the exact steps since it has been some years.

I have added the comment based on your suggestion.

I will leave this issue open. Maybe someone interested can help reproducing the calculations.

Best,

@moffile
Copy link
Author

moffile commented Sep 11, 2022 via email

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