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

Feature/adds neutral stake mech control #130

Merged
merged 112 commits into from
Mar 2, 2025

Conversation

MaxxWilson
Copy link
Collaborator

@MaxxWilson MaxxWilson commented Mar 2, 2025

PR Summary

PR Link: Link

Description

Adds VEX OS style current limiting and Neutral Stake arm control

Reviewers

Tag reviewers.


Changelog

  • Fixes bug where conveyor and ground pickup were included with drivetrain current limiting
  • Adds current limiting logic in ghost_control to mimic the V5 Brain.

Testing

Automatic

Manual

  • Tested and confirmed on robot hardware, by printing currents to brain screen and cross-referencing with the Purdue calculator.

Documentation

- Link any relevant documentation

Omega Jerry and others added 30 commits January 1, 1970 00:00
fix gains
good move to pose
both tested on hardware
adds intake hook rejection
…-GHOST/VEXU_GHOST into hw-test/move-to-pose-tuning-2-18-25
…-GHOST/VEXU_GHOST into hw-test/move-to-pose-tuning-2-18-25
…-GHOST/VEXU_GHOST into hw-test/move-to-pose-tuning-2-18-25
Copy link
Collaborator

@JakeWendling JakeWendling left a comment

Choose a reason for hiding this comment

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

lgtm

@JakeWendling JakeWendling merged commit 3295331 into develop Mar 2, 2025
1 check passed
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.

3 participants