-
Notifications
You must be signed in to change notification settings - Fork 19
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
Documentation: Acceleration registers are read-only + remove one TODO comment #334
Conversation
All values in bytes. Δ Delta to base
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating the documentation
Can't promise it right now 😄 I will take a look at the code to get a better idea of how much work is needed. I suspect the major thing to solve in this feature is how to handle changing the motor settings at any time. Perhaps we can use the infrastructure we already use for changing Stealth mode. EDIT: hmm it looks quite easy. Doesn't have anything to do with motor settings. |
Created #337 I don't think it is more complex than this 👀 |
The registers are now in PROGMEM (since 15 months ago), so this TODO is resolved. The TODO was added 2 years ago.
Automated Test Code Coverage ReportView details...
TOTAL: 2727 lines of code, 2030 lines executed, 74% covered. |
Closing in favor of #341 |
No description provided.