Skip to content

Comments

AutoChooser, more (not all) paths, and auto sequences#70

Open
michaelk036 wants to merge 28 commits intomainfrom
MK-path-six
Open

AutoChooser, more (not all) paths, and auto sequences#70
michaelk036 wants to merge 28 commits intomainfrom
MK-path-six

Conversation

@michaelk036
Copy link
Contributor

No description provided.

@michaelk036 michaelk036 changed the title Mk path six AutoChooser and more paths Feb 17, 2026
@michaelk036 michaelk036 marked this pull request as draft February 17, 2026 23:15
@michaelk036 michaelk036 changed the title AutoChooser and more paths AutoChooser, more (not all) paths, and auto sequences Feb 20, 2026
@michaelk036 michaelk036 marked this pull request as ready for review February 20, 2026 23:06
@michaelk036
Copy link
Contributor Author

This does not have updated paths, it does have updated autochooser and sequences.

commandMap.put(new AutoEvent(AutoAction.SHOOT, FieldLocation.MIDDLE),
new MidShoot(subsystem, auto, shooter, shootstate, hopper, feeder));

Optional<DriverStation.Alliance> alliance = DriverStation.getAlliance();
Copy link
Contributor

Choose a reason for hiding this comment

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

this will not work because when the map gets populated we don't know the alliance color.
let's discuss possible solutions before implementing.

@@ -0,0 +1,40 @@
package frc.robot.commands.auto;
Copy link
Contributor

Choose a reason for hiding this comment

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

All the autonomous commands should include moving turret+angler down and then resetting their encoder. This can be done in parallel with the first trajectory.

* joysticks}.
*/
private void setUpAutoFactory() {
if (drivebase == null) {
Copy link
Contributor

Choose a reason for hiding this comment

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

what is this for? is it for testbed? if so make it clearer either by adding a comment or by using the same condition that decides whether to instantiate the drivetrain or not.

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.

4 participants