Skip to content

Created a Climb command group#49

Open
dsakai4 wants to merge 3 commits into
productionfrom
Issue-46-Climb-Command-Group
Open

Created a Climb command group#49
dsakai4 wants to merge 3 commits into
productionfrom
Issue-46-Climb-Command-Group

Conversation

@dsakai4
Copy link
Copy Markdown
Contributor

@dsakai4 dsakai4 commented Jan 25, 2025

Added a Climb command and a Climb command group that calls the Funnel Open command and the Climb command sequentially. Additionally edited javadocs for Funnel commands.

@dsakai4 dsakai4 self-assigned this Jan 25, 2025
@dsakai4 dsakai4 changed the title Created a Climb command group. Created a Climb command group Jan 25, 2025
@dsakai4 dsakai4 force-pushed the Issue-46-Climb-Command-Group branch from fddd753 to 8a01024 Compare January 25, 2025 23:31
@dsakai4 dsakai4 linked an issue Jan 26, 2025 that may be closed by this pull request
@jasonsoohoo
Copy link
Copy Markdown
Member

The OpenFunnel command needs to have its isFinished method rewritten instead of just returning false. In a sequential command group commands won't move on to the next one until the prior command finishes. So in the case of the one you wrote, OpenFunnel will never be finished so Climb will never start.

Copy link
Copy Markdown
Member

@jasonsoohoo jasonsoohoo left a comment

Choose a reason for hiding this comment

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

See comment above.

@dsakai4 dsakai4 force-pushed the Issue-46-Climb-Command-Group branch from 8a01024 to 5d04dac Compare January 29, 2025 04:12
Comment thread Big-Bad-Wolf-2025/src/main/java/frc/robot/commands/FunnelOpenCommand.java Outdated
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.

climb command group

3 participants