Add intake jostling system with sine wave#101
Open
enderpichu wants to merge 6 commits into
Open
Conversation
enderpichu
commented
Mar 28, 2026
enderpichu
commented
Contributor
- Has NOT been tested on real hardware (none of our intakepivots have worked lol
- Has not been "tested" in sim- intakepivot in sim is buggy and kinda just flops over
- the only testing done has been a motion graph which seems somewhat reasonable for our intakepivot timing -ish
Create the jostleintake command, comment out mentions of previous terse command factory in robotcontainer and intakepivot.
I think it is because of how robot viz and that are setup, need to try and implement a fix for that
CHANGE values later Tune for correct jostle, according to schoen and me this current timing seems reasonable for our intake.
enderpichu
requested review from
Nathanael7639,
gavinskycastle and
jonathandao0
Collaborator
|
this was merged to the oc branch already so i think we can just merge that |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.