Skip to content

add read me#148

Merged
riley-jen merged 4 commits into
mainfrom
riley/add-read-me
May 4, 2026
Merged

add read me#148
riley-jen merged 4 commits into
mainfrom
riley/add-read-me

Conversation

@riley-jen

Copy link
Copy Markdown
Member

after a very long time of being bugged to do this...
augment generated btw, plz check for mistakes

@augmentcode

augmentcode Bot commented Apr 16, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR replaces the template README with a detailed overview of Team 6418’s 2026 robot codebase.

Changes:

  • Renames the project header and describes major subsystems (swerve drivetrain, intake, shooter, vision)
  • Documents key architectural patterns (command-based structure, hardware IO abstraction, command factory)
  • Adds a project tree to help new contributors find code quickly
  • Explains constants organization and typical configuration items (IDs, limits, PID/FF, Motion Magic)
  • Summarizes autonomous approach with PathPlanner + named commands
  • Adds build/test/deploy commands and lists major dependencies/documentation links

Technical Notes: README calls out CTRE Phoenix 6 swerve, PhotonVision AprilTag pose estimation, and MapleSim-based swerve simulation support.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. 4 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread README.md
- **Column**: Triple rollers that feed game pieces to shooter

**Shooter**
- Three independent motors (Influencer, Follower, Third) with real-time velocity control

@augmentcode augmentcode Bot Apr 16, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

README.md:24 — The shooter motors aren’t “independent” in the current implementation: ShooterSubsystem configures FOLLOWER and THIRD to followMotor(m_influencerIO, false), so only the influencer is directly commanded.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

Comment thread README.md Outdated

**Shooter**
- Three independent motors (Influencer, Follower, Third) with real-time velocity control
- **Shoot on the Move**: Dynamic velocity adjustment while translating

@augmentcode augmentcode Bot Apr 16, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

README.md:25 — The README says Shoot on the Move does “dynamic velocity adjustment while translating,” but RobotCommandFactory comments/logic indicate SOTF keeps shooter velocity the same and adjusts drivetrain aiming angle instead ("only the angle changes").

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

Comment thread README.md
src/main/java/frc/robot/
├── Robot.java # Main robot class
├── RobotContainer.java # Robot initialization and command binding
├── RobotCommandFactory.java # Factory for creating robot-wide commands

@augmentcode augmentcode Bot Apr 16, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

README.md:49 — The project structure shows src/main/java/frc/robot/RobotCommandFactory.java, but RobotCommandFactory is actually located under src/main/java/frc/robot/subsystems/RobotCommandFactory.java in this repo.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

Comment thread README.md

## Autonomous Routines

Autonomous commands are defined in `Autos.java` and integrated with **PathPlanner** for trajectory generation:

@augmentcode augmentcode Bot Apr 16, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

README.md:103 — “Autonomous commands are defined in Autos.java” looks misleading here: autos are primarily selected/built via PathPlanner AutoBuilder.buildAutoChooser(...) and src/main/deploy/pathplanner/autos/*.auto, while commands/Autos.java appears to be the default example/template file.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@moreheadm moreheadm left a comment

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.

approving under duress, not enough Emojis

@riley-jen riley-jen merged commit ebda94d into main May 4, 2026
1 check passed
@riley-jen riley-jen deleted the riley/add-read-me branch May 4, 2026 06:48
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