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

Documentation #69

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Documentation #69

wants to merge 4 commits into from

Conversation

realtrev
Copy link
Contributor

@realtrev realtrev commented Apr 9, 2023

Purpose: to add documentation to files that were confusing or lacked documentation. Unnecessary code was also removed and cleaned up.

Files were reformatted to use 4 spaces instead of two.

Documentation added to the following files, as well as any other comments:

  • KeyValue.java
  • Kinematics.java
  • Position2D.java
  • SPIKE293Utils.java
  • SmoothControl.java
  • SpikeBoard.java
  • AdjustArm.java
  • ArcadeDrive.java
  • AutoBalance.java
  • BumpDrive.java
  • CalibrateExtender.java
  • CalibratePivot.java
  • DriveTo.java
  • DriveToBalance.java
  • ForzaDrive.java
  • MoveArm.java
  • MoveClaw.java
  • RCFDrive.java
  • ResetKinematics.java
  • Rotate.java
  • SequentialAutoCommand.java: Removed parts that weren't being used, namely leave community and balance.
  • SetArm.java
  • SetClaw.java
  • SetClawForTime.java
  • TrackTarget.java
  • ZeroArm.java
  • Arm.java
  • Claw.java
  • Drivetrain.java
  • Targeting.java

Files reformattted:

  • WriteToCSV.java

Files deleted:

  • DriveBackwards.java: Unnecessary. Was only being used in SAC and was removed.

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

Successfully merging this pull request may close these issues.

2 participants