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

Add TurnToAngle #21

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Add TurnToAngle #21

wants to merge 7 commits into from

Conversation

frc6897
Copy link
Contributor

@frc6897 frc6897 commented Aug 20, 2022

Part of #13. Added a TurnToAngle command which turns the robot to the specified angle. Useful for autonomous routines where tight turns are needed.

Edit 9-23-22: Added NITRO button! It's set as triangle. Resolves #9.

@frc6897 frc6897 added subsytem:drive This issue is related to drive subsystem feature This issue adds a new feature labels Aug 20, 2022
@siddhxrth siddhxrth linked an issue Aug 21, 2022 that may be closed by this pull request
Copy link
Member

@siddhxrth siddhxrth left a comment

Choose a reason for hiding this comment

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

Wow, this is some beautiful code! I wonder who wrote it... he must be a genius!! 😍

Copy link
Contributor

@jgoldberger26 jgoldberger26 left a comment

Choose a reason for hiding this comment

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

After output is clamped, this is ready to be tested and then merged.

However, try and keep commits as relevant as possible. For example, many lines of code have been unnecessarily formatted, m_crossButton should have been changed in a refactoring issue, and fixing the field image should have been given its own issue. This will eventually lead to merge conflicts if everything is not kept isolated.

src/main/java/frc/robot/commands/drive/TurnToAngle.java Outdated Show resolved Hide resolved
src/main/java/frc/robot/RobotContainer.java Show resolved Hide resolved
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 7, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 9 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue adds a new feature subsytem:drive This issue is related to drive subsystem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create TurnToAngle command Add Turbo Button
4 participants