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.

Sign up for GitHub

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

Jump to bottom

Intake #15

Merged
merged 25 commits into from
Mar 21, 2024
Merged

Intake #15

merged 25 commits into from
Mar 21, 2024

Conversation

karmalover-ca
Copy link
Collaborator

I dont feel like rebasing

karmalover-ca and others added 25 commits March 1, 2024 20:05
@karmalover-ca
adds motor controllers for intake to robotmap
00fa9a8
@karmalover-ca
adds skeleton code for intake subsystems
818f5e2
@karmalover-ca
adds command to spin arm temp
ff77fc0
@karmalover-ca
adds wrist pid things, function to move arm piston
f318f9b
@karmalover-ca
removes spinArm button, adds some pid things, makes it so you can spi…
66ab5dd 
…n claw etc
@karmalover-ca
makes compressor use analog switch, makes arm actually work
dbffd75
@karmalover-ca
tunes arm to move up and down smoothly
a27a98b
@gbuckholtz
Some wrist subsystem mehods that need implementing
8371552
@gbuckholtz
An arm movement to be implemented
f6f05b9
@gbuckholtz
First Pass for Intake Command Done
1752219
@gbuckholtz
added anp, trap, and transport command groups
c091481
@karmalover-ca
makes wrist move with pid, and tunes pid number to be fast
98ae916
@gbuckholtz
added controls for operator to OI and RobotContainer for the IntakeCo…
8e0cda8 
…mmandGroup, TransportCommandGroup, AmpCommandGroup, and TrapCommandGroup
@gbuckholtz
added commands for intake and shooting
e61a34a
@gbuckholtz
added commands for climbing and added needed functions to Climber sub…
2634544 
…system
@gbuckholtz
Initial Climb code
0598052
@gbuckholtz
Merge remote-tracking branch 'origin/intake' into commands_initial_cu…
0d9ea88 
…t plus climber tuning
@gbuckholtz
oops, here is the climber tuning
50e6160
@gbuckholtz
Merge pull request #10 from LakeEffectRobotics/commands_initial_cut
8baa679 
Wrist methods that need implementing
@gbuckholtz
removed the climbCommand. we are using Commands.runonce with a lamda …
2015535 
…instead
@gbuckholtz
removed and import dependency
00d367f
@gbuckholtz
Added subsystem requirements to the lambda in the climb button bindings
03407f3
@karmalover-ca
changes pid values to match with more tunes pot readings
3619aaa
@karmalover-ca
tunes wrist to be even better
a675e92
@karmalover-ca
Merge branch 'master' into intake
f16fe1d
@karmalover-ca
Copy link
Collaborator Author

the merge from master is just drive input cleaning and gyro stuff

gbuckholtz
gbuckholtz approved these changes Mar 20, 2024
View reviewed changes
Copy link
Contributor

@gbuckholtz gbuckholtz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@gbuckholtz gbuckholtz merged commit da9aafe into master Mar 21, 2024
2 checks passed
@karmalover-ca karmalover-ca deleted the intake branch March 21, 2024 00:02
@karmalover-ca karmalover-ca mentioned this pull request Mar 21, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gbuckholtz gbuckholtz gbuckholtz approved these changes

@gregk27 gregk27 Awaiting requested review from gregk27

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@karmalover-ca @gbuckholtz