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.

Sign up for GitHub

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

Jump to bottom

Enable velocity control and write calibrations as pbs #48

Merged
merged 1 commit into from
Jul 8, 2022
Merged

Enable velocity control and write calibrations as pbs #48

merged 1 commit into from
Jul 8, 2022

Conversation

gbalke
Copy link
Member

@gbalke gbalke commented Jul 7, 2022

This is a two part commit.

  1. Switches the calibration storage system to use protobufs
  2. Finishes off velocity control with gains for small motors

@gbalke gbalke requested a review from brentyi July 7, 2022 19:48
@gbalke gbalke force-pushed the velocity_control branch from 84c6cbf to ebd106f Compare July 7, 2022 19:54
@gbalke
Copy link
Member Author

gbalke commented Jul 7, 2022

This PR should resolve #31 save for the testing/validation.

@gbalke
Enable velocity control; write calibrations as pbs
21ff087 
Signed-off-by: Greg Balke <gbalke@berkeley.edu>
@gbalke gbalke force-pushed the velocity_control branch 2 times, most recently from e8db2be to 21ff087 Compare July 7, 2022 20:20
@gbalke gbalke merged commit 906a962 into master Jul 8, 2022
@gbalke gbalke deleted the velocity_control branch July 8, 2022 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brentyi brentyi Awaiting requested review from brentyi

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@gbalke