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

make intake #6

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

make intake #6

wants to merge 8 commits into from

Conversation

jwannebo3524
Copy link
Contributor

Subsystem: Intake

Description:

Added intake subsystem

Tested: no

Collaborators: @dimwitt @dimwitt71 Nathan, idk which one


Before submitting:

  • Review that your code meets our coding standards
  • Confirm there is a green ✔️ (not a red ❌) next to your last commit's date
  • Replace all text above with appropriate descriptions
  • Remove header and footer from this template
  • Add @smccrorie and @fruzyna as reviewers

Copy link
Member

@fruzyna fruzyna left a comment

Choose a reason for hiding this comment

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

Looks like it would work, just needs some touch ups

@smccrorie
Copy link
Contributor

One more change request - I think we're going to need to have the intake ingest at like 10% power whenever it has a game piece in it's jaw to ensure the game piece doesn't fall out. So something where if neither trigger is pressed and the last action was ingesting, the motor runs in at 10%. But if the last action was outtaking, the motor stops.

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