-
Notifications
You must be signed in to change notification settings - Fork 139
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
Clarify instructions in README + update default gamepad velocity values #112
Conversation
@gfosmire since you still haven't reviewed this, pulling you off and adding @apollokit |
Right off the bat, of course, this'll need to be updated to reflect the changes from #108 |
… into update_readme_configuring Resolve merge conflicts by simply moving changes to appropriate readme locations
@apollokit resolved the merge conflict, ready for review |
Sorry for the delay, will get to this this week |
Co-authored-by: Kit Kennedy <kitkennedy8@gmail.com>
Addressed your suggestions. Let me know if it makes more sense and feel free to make another suggestion |
Cool, looks good. |
FYI policy we've used in this repo is to let the assignee or PR opener do the merge. For this one it doesn't matter but it can cause trouble when there's dependent PRs and so on that the reviewer isn't always aware of |
Gotcha, noted. |
Added some extra clarifications + verbosity to the main README
The default velocities for the default drive motor configuration (31rpm) was also incorrect with the velocities for turbo being way too high.