Skip to content

update ParameterizedIntakeSubsystem to take in Motor instead of PIDmotor#67

Merged
spderman3333 merged 4 commits into
mainfrom
katy/updateintaketousemotor
Oct 6, 2025
Merged

update ParameterizedIntakeSubsystem to take in Motor instead of PIDmotor#67
spderman3333 merged 4 commits into
mainfrom
katy/updateintaketousemotor

Conversation

@katyyeh527
Copy link
Copy Markdown
Contributor

No description provided.

kcooney and others added 3 commits September 24, 2025 20:53
This reduces duplication, and allows us to not have to add vendordeps for
indirect dependencies.
This reduces the number of dependencies pulled in by :lib, which reduces how
much code has to be deployed to the robot.
@kcooney kcooney changed the base branch from kcooney/move-fake-pid-motor to main October 3, 2025 03:18
@kcooney kcooney requested a review from spderman3333 October 3, 2025 03:18
Copy link
Copy Markdown
Member

@spderman3333 spderman3333 left a comment

Choose a reason for hiding this comment

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

lgtm

@spderman3333 spderman3333 merged commit 5ce88dd into main Oct 6, 2025
1 check passed
@spderman3333 spderman3333 deleted the katy/updateintaketousemotor branch October 6, 2025 03:42
kcooney added a commit to Prospect-Robotics/Robot2025 that referenced this pull request Oct 15, 2025
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.

ParameterizedIntakeSubsystem should depend on Motor, not PIDMotor

3 participants