Skip to content

Chore: Update Robot Controller. See additional notes below.#4

Merged
dguisti merged 4 commits intoVCSilverCircuits:mainfrom
awdev1:main
Jun 29, 2025
Merged

Chore: Update Robot Controller. See additional notes below.#4
dguisti merged 4 commits intoVCSilverCircuits:mainfrom
awdev1:main

Conversation

@awdev1
Copy link
Contributor

@awdev1 awdev1 commented Jun 21, 2025

Overview

This PR adds a capped speed control to the OutreachTele OpMode to ensure safer driving during outreach events. Extra libs were removed and "how to drive robot" instructions were added.

Changes

  • Updates Robot Controller from 10.1.1 to 10.2
  • Introduced driveSpeed = 0.5 to limit motor power so we don't break things on impact
  • Added elemetry with clear movement "how-to" instructions
  • Removed libs to override telemetry since I didn't wish to bother around with it
  • Introduced FTC Dashboard Option for speed control.

Disabled certain OPModes for safety. 50% speed to reduce possible damage to humans or materials. Git is hiding most of the diff for no reason, but the FTCRobotController folder is new

Disabled certain OPModes for safety. 50% speed to reduce possible damage to humans or materials
@awdev1 awdev1 self-assigned this Jun 21, 2025
@awdev1 awdev1 added the enhancement New feature or request label Jun 21, 2025
Disabled certain OPModes for safety. 50% speed to reduce possible damage to humans or materials
@awdev1
Copy link
Contributor Author

awdev1 commented Jun 21, 2025

@awdev1 awdev1 requested review from Midnight-Design and removed request for Midnight-Design June 21, 2025 21:04
Copy link
Contributor

@dguisti dguisti left a comment

Choose a reason for hiding this comment

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

Looks good. Might want to hook into core later for maintainability, but for outreach this is fine.

@awdev1
Copy link
Contributor Author

awdev1 commented Jun 21, 2025

Looks good. Might want to hook into core later for maintainability, but for outreach this is fine.

Core was removed in favor of User Instructions on how to use the robot. BaseTeleOp was overriding my telemetry for that and I didn't want to modify core.

@awdev1 awdev1 requested a review from Midnight-Design June 21, 2025 21:08
Copy link
Contributor

@TuckerzKnight TuckerzKnight left a comment

Choose a reason for hiding this comment

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

Um it looks like there was a comment in there referring to the REV motors used (line 28), that information is not relevant to the following code could you please remove that

@awdev1
Copy link
Contributor Author

awdev1 commented Jun 29, 2025

Um it looks like there was a comment in there referring to the REV motors used (line 28), that information is not relevant to the following code could you please remove that

Undertow V1 uses REV Motors unlike GoBilda in V2

@awdev1 awdev1 removed the request for review from Midnight-Design June 29, 2025 05:31
@dguisti dguisti merged commit 931e905 into VCSilverCircuits:main Jun 29, 2025
@awdev1
Copy link
Contributor Author

awdev1 commented Jun 29, 2025

Thanks!

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants