Skip to content

Comments

diags#55

Open
Mobby42 wants to merge 9 commits intomainfrom
JK_diags
Open

diags#55
Mobby42 wants to merge 9 commits intomainfrom
JK_diags

Conversation

@Mobby42
Copy link
Contributor

@Mobby42 Mobby42 commented Feb 16, 2026

No description provided.

@@ -0,0 +1,1203 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

Somebody pushed the layout a few days ago, just want to make sure this includes these changes. Also - seems like there's a lot of debug stuff here. We may want to clean it up.



//Diags
public static final double HOPPER_DIAGS_ENCODER = 1;
Copy link
Contributor

Choose a reason for hiding this comment

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

should we add encoder diag for turret, angler, intakeDeployer?


Robot.getDiagnostics()
.addDiagnosable(
new DiagSparkMaxEncoder(
Copy link
Contributor

Choose a reason for hiding this comment

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

add limit switches to climber. Top and Bottom.

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.

2 participants