Skip to content

Comments

No Hitting Trench In Autonomous#50

Merged
Trigon5990 merged 3 commits intomainfrom
no-hitting-trench-in-autonomous
Feb 22, 2026
Merged

No Hitting Trench In Autonomous#50
Trigon5990 merged 3 commits intomainfrom
no-hitting-trench-in-autonomous

Conversation

@Trigon5990
Copy link
Contributor

No description provided.

Trigon5990 and others added 2 commits February 22, 2026 21:01
Co-Authored-By: Nummun14 <142012009+Nummun14@users.noreply.github.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 22, 2026

Walkthrough

The PR refactors autonomous shooting logic in GeneralAutonomousCommands: getShootAtHubWhileDrivingCommand now selects between three paths (hoodless, hub, over-trench) with runtime logging, and is gated by alliance-zone checks plus a new public isInTrench() predicate. It adds a private isAngleCloserTo180(Rotation2d) helper and getPrepareForShootingWithoutHoodCommand() returning a fixed-87° hood command. AutonomousGenerator gains a log entry "Autonomous/IsInTrench" that records isInTrench(), and AutonomousConstants.SCORING_TIMEOUT_SECONDS is adjusted from 3.5 to 3.7.

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to evaluate whether the description relates to the changeset. Add a description explaining the motivation and implementation of the trench-avoidance logic, such as mentioning the new isInTrench() helper and its integration into the shooting path decision logic.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title 'No Hitting Trench In Autonomous' directly describes the main change: preventing the robot from entering the trench during autonomous mode, which is the core purpose of the isInTrench() check and conditional logic added throughout the changeset.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 5

Co-Authored-By: Nummun14 <142012009+Nummun14@users.noreply.github.com>
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

@Trigon5990 Trigon5990 merged commit c76deaa into main Feb 22, 2026
4 checks passed
@Trigon5990 Trigon5990 deleted the no-hitting-trench-in-autonomous branch February 22, 2026 19:41
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