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

63 make safety logic for elevator and arm better #64

Open
wants to merge 30 commits into
base: develop
Choose a base branch
from
Open

63 make safety logic for elevator and arm better #64

wants to merge 30 commits into from

Conversation

qwertycloudhub
Copy link
Collaborator

Pull Request Template

Type of Change

  • Bug Fix
  • New Feature
  • Code Refactor
  • Documentation Update
  • Other (please specify):

Subsystem(s) Modified

  • Wrist
  • Arm and elevator safety logic

Description of Changes

Made arm and elevator movement smoother
Added wrist
Integrated wrist into safety logic

Additional Context

Checklist

  • Code follows team's coding standards
  • Comments added/updated where needed
  • Changes have been tested in simulation (if applicable)
  • Changes have been tested on the robot (if applicable)

Screenshots/Videos

taran-duba and others added 20 commits January 10, 2025 17:19
@taran-duba
added WristSubsystem
db681ec
@taran-duba
Merge branch 'develop' into 14-wrist-subsystem
f0e8bdb
@taran-duba
Merge branch 'develop' into 14-wrist-subsystem
60dc408
@taran-duba
Merge remote-tracking branch 'origin/14-wrist-subsystem' into 14-wris…
55e8593 
…t-subsystem
@taran-duba
fixed gradle violations
01b5d67
@taran-duba
Merge branch 'develop' into 14-wrist-subsystem
714b687
@taran-duba
Merge branch 'develop' into 14-wrist-subsystem
e0170bb
@taran-duba
Merge remote-tracking branch 'origin/14-wrist-subsystem' into 14-wris…
22858db 
…t-subsystem
@taran-duba
switch encoder units to rotations; updated config
a723085
@taran-duba
fixed gradle violations
ae4633c
@taran-duba
fixed gear ratio numbers
b0c3419
@taran-duba
added gear ratios in TalonFXConfiguration
ec0c96b
@taran-duba
fixed gradle violations
e8ab8a9
@yetirobotics
Merge branch 'develop' of https://github.com/Yeti-Robotics/reefscape-…
cf8bb72 
…2025 into 14-wrist-subsystem
@yetirobotics
reviving the wrist
6b51baf
@taran-duba
tuned wrist in simulation
a5c52ab
@taran-duba
Merge branch 'develop' into 14-wrist-subsystem
b5558f6
@qwertycloudhub
fix elevator-arm safety logic to prevent unnecessary movement
421aaf7
@taran-duba
refactored wrist and moved under coral
11eddb8
@qwertycloudhub
merged wrist logic
c58c0dc
@qwertycloudhub qwertycloudhub linked an issue Mar 5, 2025 that may be closed by this pull request
Make safety logic for elevator and arm better #63
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@samperlmutter samperlmutter Awaiting requested review from samperlmutter

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.

Make safety logic for elevator and arm better
3 participants
@qwertycloudhub @taran-duba @yetirobotics