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

Nate/chaos auto #44

Closed
wants to merge 8 commits into from
Closed

Nate/chaos auto #44

wants to merge 8 commits into from

Conversation

NateLydem
Copy link
Contributor

Adding a chaos auto for all notes in center

@NateLydem NateLydem requested a review from a team as a code owner April 4, 2024 00:34
Copy link
Contributor

@WeilSimon WeilSimon 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 this should work quite well but the command will crash the code at the moment.

src/main/deploy/choreo/SourceLanephgfedChaos.traj Outdated Show resolved Hide resolved
PathHelper.fromChoreoPath("SourceLanephgfedChaos.4"),
PathHelper.fromChoreoPath("SourceLanephgfedChaos.5")
);
addCommands(
Copy link
Contributor

Choose a reason for hiding this comment

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

The best way to run this as intended is probably to have a parallel command group with the path, intake and feed, and a prepare shooter command with the same angle as a hub shot but with probably 1000 rpm on the left and right flywheels. This rpm can also be tuned in the pits tomorrow.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The shooter is only going to be used to get the note away from where it was picked up as fast as possible, without shooting out of the field. Should I not use prepare shooter and have a low rpm instead?

@rutmanz rutmanz deleted the branch working/camas3 April 9, 2024 05:40
@rutmanz rutmanz closed this Apr 9, 2024
@rutmanz
Copy link
Member

rutmanz commented Apr 9, 2024

Replaced by #46

@rutmanz rutmanz mentioned this pull request Apr 9, 2024
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