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

Document the drive classes #1109

Closed
calcmogul opened this issue Jan 18, 2021 · 3 comments · Fixed by #1132
Closed

Document the drive classes #1109

calcmogul opened this issue Jan 18, 2021 · 3 comments · Fixed by #1132

Comments

@calcmogul
Copy link
Member

calcmogul commented Jan 18, 2021

It should cover the following:

  1. What drive inverse kinematics classes are available (DifferentialDrive, MecanumDrive)
  2. What inverse kinematic methods each class provides
  3. What RobotDriveBase.setDeadband() does (adds a deadband to joystick values while rescaling the passthrough region so there isn't an output discontinuity; also note that users can square the input or apply other transformations to the input and the deadband will compose with it)
  4. What RobotDriveBase.setMaxOutput() does
@jasondaming
Copy link
Member

This article seems out of place under actuators but I think we can add this stuff to it rather than create a new article. We really need to rethink the structure of Basic Programming as it is a catchall that is hard to find what you want / need.

https://docs.wpilib.org/en/stable/docs/software/actuators/wpi-drive-classes.html

@Daltz333
Copy link
Member

Daltz333 commented Jan 21, 2021

Ideally I think this would be in the zero-to-robot step-4 section as part of a "programming your robot" tutorial

@jasondaming
Copy link
Member

I think it is important but I disagree. Zero to Robot gets them running. This is optimization, I think it is good optimization that teams should do but that is a slippery slope of do we include auto tutorial or a joystick howto or....which are all optimizations.

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

Successfully merging a pull request may close this issue.

3 participants