BALLING into main - #263
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a new 'Balling' mode for basketball hoop scoring, updates CTRE Phoenix 6 and PhotonLib dependencies, and implements a currently-disabled swerve module slip detection algorithm. It also refines controls and turret bindings, and disables field boundary clamping when Balling mode is active. Feedback on the changes highlights two critical issues: first, ballingDistanceData is incorrectly populated into standard maps instead of balling-specific maps in LauncherConstants.java, which will cause lookup failures; second, a potential NullPointerException exists in Controls.java because s.flywheels is not checked for null before retrieving its target speed.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
No description provided.