-
Notifications
You must be signed in to change notification settings - Fork 3
Robin amp branch #10
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
Open
kitzoyan
wants to merge
68
commits into
Team7520:master
Choose a base branch
from
Team7520:robinAmpBranch
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Robin amp branch #10
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Needs PID and better controls
There is no default command for IntakeRollers
To-Do: Add Smartmotion for arm positions
Sync Fork
Intake 2024
Adds a couple forgotten lines of code
Adding Climber Code
Auto should be working not tested yet
Additional Auto: Speaker Source Side 1Note Shot to Note8 Parking No I…
Added new two note auto, From Speaker ampside to note 1
Fix climber auto drifting issue
added auto: robot stay after 2nd note
SpeakerS.Note3 Auto
Update Climber.java
4note auto with same speed but more direct paths
Added 2 note SpeakerS to Note8 to SpeakerS
…te2.SpeakerC.Note5.SpeakerC (Untested)
4 Note Auto
# Conflicts: # src/main/deploy/pathplanner/paths/Close Mid to Mid.path # src/main/deploy/pathplanner/paths/Mid To Close Mid.path # src/main/deploy/pathplanner/paths/Mid To Close Top.path # src/main/deploy/pathplanner/paths/SpeakerC.Note3.path # src/main/java/frc/team7520/robot/RobotContainer.java
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Created a new subsystem called amp. Treat it like every other subsystem, same format basically copy and pasted from other subsystems. Added constants for it, hoping to get it to have PID but not implemented (but base code for that is already there). Currently, amp mech is controlled by the operator Dpad during teleop. The resting position is 0, and amp position is +135 encoder values. I've lowered some PID constants (for the amp) but when it wasn't working... should be easy integration though.