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

Restructure #18

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

Restructure #18

wants to merge 64 commits into from

Conversation

acabey
Copy link
Contributor

@acabey acabey commented Mar 31, 2017

  • Refactored most commands to use parameters in constructor
  • Includes ultrasonic implementation
  • PID Controller should be refactored

Larry Tseng and others added 30 commits March 17, 2017 17:17
Larry, don't look at this code
Analog implementation only
Integrated analog into OI
Completed several javadocs 
Closed several todos
Robot features
Packages
Naming conventions
Added social media links
Conflicts:
	src/com/team1757/robot/RobotMap.java
Rotate commands are able to easily use either constructor parameters or
pull from SmartDashboard
Moved SmartDashboard operations into GetStatus (from other commands)

Improved javadocs
Improved javadocs for shooter

Removed ring lights from vision subsystem
Removed useless commands (accelerometer, reset PID, RotateDegrees)

Renamed RotateDegreesShortest to RotateDegrees
acabey and others added 23 commits April 1, 2017 12:19
Removed unused classpath variables as all libraries are localized
Allows vision processing to work with low exposure and driver view with
high
Tried adding a delay between exposure change and processing - GRIP still
fails to detect contour
Inaccurate reporting bug unsolved
intended and runs test command when first activated.
Velocities inverted. Pre match 3
Completely functional autonomous
Completely functional drop gear override
Drop gear override enabled by default

End of day 1 at District Championship
Camera still reporting error
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants