Skip to content
This repository has been archived by the owner on Apr 29, 2024. It is now read-only.

Rework the DriveTrain class #12

Open
RED-M0CKING-LINE opened this issue Nov 22, 2019 · 2 comments
Open

Rework the DriveTrain class #12

RED-M0CKING-LINE opened this issue Nov 22, 2019 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@RED-M0CKING-LINE
Copy link
Owner

Go though and make general improvements. add in a sinusoidal curve as well with the same features as the driveExp() method.
add default values for the driveExp() method. required parameters should be drive and turn speeds, whereas optional values can be modification of the X and Z exaggeration values, the deadzone, and the place value of speed to pass.

also clip the values passed into the motors rounded to the thousandths place for each drive function. consider a private function that is designed to handle clipping output values, applying the dead zone, and passing values to the motors.
If this feature is implemented, move the curves to their own function class and have the functions in the DriveTrain class refer to those curves with default values already implemented. This would allow the curves to be applied to any feature of the robot quickly and saving code.

@RED-M0CKING-LINE RED-M0CKING-LINE self-assigned this Nov 22, 2019
@RED-M0CKING-LINE RED-M0CKING-LINE added the enhancement New feature or request label Nov 22, 2019
@RED-M0CKING-LINE
Copy link
Owner Author

also, impliment the option between arcade drive or curvature drive :)

DRIVE.curvatureDrive(xSpeed, zRotation, isQuickTurn);

@RED-M0CKING-LINE
Copy link
Owner Author

figure out what the input dead zone should be so that the motors cannot be stalled out. This should be done by calculating something XD

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant