Skip to content

Pit scouting#8

Merged
marisa07 merged 7 commits intomainfrom
pit-scouting
Mar 5, 2026
Merged

Pit scouting#8
marisa07 merged 7 commits intomainfrom
pit-scouting

Conversation

@marisa07
Copy link
Copy Markdown

@marisa07 marisa07 commented Mar 5, 2026

No description provided.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 5, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 312b8629-4479-405c-9081-ac6b89ce0cad

📥 Commits

Reviewing files that changed from the base of the PR and between d8fd6ca and 2599646.

📒 Files selected for processing (2)
  • 2026/rebuilt_pit_scouting.js
  • resources/js/googleSheets.js

Summary by CodeRabbit

Release Notes

  • New Features
    • Pit scouting form enhanced with climbing capability levels (L1, L2, L3), shooting method documentation, battery count, and photo upload options.
    • Added fields for robot dimensions and fuel hopper capacity.
    • Expanded text areas for detailed autonomous and climbing strategy feedback.
    • Simplified drivetrain selection options for easier data entry.

Walkthrough

The changes update a pit scouting form configuration by introducing new fields including Length, Hopper capacity, Ground Intake, Battery count, and three climb levels, while removing legacy fields like Swerve Ratio and Drivetrain Motor. Drivetrain options are simplified to Swerve, Tank, and Other. Text fields for climb descriptions and shooting mechanics are added. Additionally, a data submission utility now prefixes pit scouting data with "PIT\t" and updates the success message.

Changes

Cohort / File(s) Summary
Pit Scouting Configuration
2026/rebuilt_pit_scouting.js
Expands pitConfig schema with new fields: Length, Hopper capacity, Ground Intake, climb levels (L1/L2/L3), climb/shooting descriptions, and battery count. Removes legacy fields (Swerve Ratio, Drivetrain Motor, fuel pickup options). Simplifies drivetrain choices and adjusts formatting for text fields.
Data Submission
resources/js/googleSheets.js
Adds conditional prefix "PIT\t" to TSV data when pitScouting flag is active. Updates success alert message from "Success!" to "Yippee!".

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • Auto send to google sheets #1 — Also modifies resources/js/googleSheets.js to change data sending logic and button behavior in pit scouting workflows.

Poem

🐰 Fresh forms with climbs so bright,
New fields dance left and right,
Length and hoppers, batteries too,
L1, L2, L3—the pit's debut!
"Yippee!" it sings when data takes flight! 🚀

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch pit-scouting

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@marisa07 marisa07 merged commit 8d44c90 into main Mar 5, 2026
3 of 4 checks passed
@marisa07 marisa07 deleted the pit-scouting branch March 5, 2026 18:21
@coderabbitai coderabbitai bot mentioned this pull request Mar 5, 2026
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.

1 participant