Skip to content
This repository was archived by the owner on Dec 18, 2025. It is now read-only.

Auto benchmark speed#163

Closed
ElliotScher wants to merge 6 commits into
feature-v3from
auto-benchmark-speed
Closed

Auto benchmark speed#163
ElliotScher wants to merge 6 commits into
feature-v3from
auto-benchmark-speed

Conversation

@ElliotScher
Copy link
Copy Markdown
Contributor

This has all the robot side GV optimizations that are needed before the season.

@ElliotScher ElliotScher requested a review from Aradig26 December 15, 2025 23:12
vision =
new Vision(
() -> AprilTagFieldLayout.loadField(AprilTagFields.k2025ReefscapeWelded),
RobotCameras.V3_POOT_CAMS);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

undo these changes

vision =
new Vision(
() -> AprilTagFieldLayout.loadField(AprilTagFields.k2025ReefscapeWelded),
RobotCameras.V3_POOT_CAMS);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we should still use V3_POOT_CAMS

@Override
public Command getAutonomousCommand() {
return autoChooser.selectedCommand();
return AutonomousCommands.speedBenchThree(drive, superstructure, intake, manipulator).cmd();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this change was for benchmarking only

"exp":"5557 RPM",
"val":581.9276791999494
"exp":"6000 RPM",
"val":628.3185307179587
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should this still be 6000RPM?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants