-
Notifications
You must be signed in to change notification settings - Fork 0
Feature/autos #23
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
base: main
Are you sure you want to change the base?
Feature/autos #23
Conversation
spellingcat
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make sure to merge main into this when you have time
| setAutoScoreReqTrue(), | ||
| waitUntilEmpty(), | ||
| setAutoScoreReqFalse()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this may or may not need to be changed depending on if we get sotm but we can wait
| setAutoScoreReqFalse()); | ||
| } | ||
|
|
||
| // feeding and intake could prob be improved |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think it's fine. only thing is the same comment from above (feed) abt not running the path twice
| // no idea what to name them | ||
| public Command getDepotScoreClimbAuto() { | ||
| final AutoRoutine routine = factory.newRoutine("Depot Score Clim Auto"); | ||
| Path[] paths = {Path.PLtoD, Path.DtoIL, Path.ILtoILM, Path.ILMtoPL, Path.PLtoCL}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
src/main/java/frc/robot/Autos.java
Outdated
| // idk sep intake and feed so action is included makes it easier for me but they use the same | ||
| // trajectories so i dont have to make new paths |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'm not really in love with this naming scheme but not your fault bc i can't really think of much better lol
| } | ||
|
|
||
| // specific paths: | ||
| // no idea what to name them |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are these the win/lose autos cassie asked for on the doc? ik we can't exactly test the open top hopper w/alpha but it looks like these paths go to depot and then neutral and then climb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.


No description provided.