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

Multiple Anthony Autos #148

Merged
merged 21 commits into from
Mar 2, 2024
Merged

Multiple Anthony Autos #148

merged 21 commits into from
Mar 2, 2024

Conversation

JohnGilb
Copy link
Contributor

@JohnGilb JohnGilb commented Mar 2, 2024

Why are we doing this?

Anthony wrote a series of useful auto commands - binding them to NeoTrellis so we can use them at runtime.
Asana task URL:

Whats changing?

Some of the bottom 8 buttons on NeoTrellis now run Anthony's autos

Questions/notes for reviewers

How this was tested

  • unit tests added
  • tested on robot
  • tested in simulator

anthonytrenh and others added 20 commits February 24, 2024 14:55
- made a PointToSpeakerCommand so that DistanceShot autos are able to aim at subwoofer
- started working on DistanceShotFromMidShootThenShootMiddleTopThenTopCenter auto program (preload, top two spikes, top center note)
- removed stop RPM from CollectSequenceCommapGroup
- getGamePieceReady() changed to confidentlyHasControlOfNote()
- updated controller rumble in IntakeCollectorCommand (not tested)
- commented out using vision assisted pose for testing auto
- made drive top and bottom subwoofer commands
- changed subwoofer bottom and top scoring location rotation in PoseSubsystem
- created SpikeTopWhiteLine and SpikeBottomWhiteLine
…op side

made two different versions of the DriveToListOfPointsCommand. One is for collecting and points at the swerve points backwards, while one is general and just drives to points (doesn't have aim at goal, so I can give set rotations instead
# Conflicts:
#	src/main/java/competition/operator_interface/OperatorCommandMap.java
#	src/main/java/competition/subsystems/pose/PoseSubsystem.java
@JohnGilb JohnGilb requested a review from a team as a code owner March 2, 2024 17:37
@JohnGilb JohnGilb merged commit e941895 into main Mar 2, 2024
1 check passed
@JohnGilb JohnGilb deleted the 4NoteAutoTesting branch March 2, 2024 17:51
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