Skip to content

Various fixes prior to release - #272

Merged
Adrianamm merged 10 commits into
frcsoftware:mainfrom
samfreund:various-fixes
Sep 11, 2026
Merged

Various fixes prior to release#272
Adrianamm merged 10 commits into
frcsoftware:mainfrom
samfreund:various-fixes

Conversation

@samfreund

@samfreund samfreund commented Sep 11, 2026

Copy link
Copy Markdown
Member

AI Disclosure: trawled through to find issues. Some changes human, some AI. All reviewed by myself.

Signed-off-by: samfreund <samf.236@proton.me>
@github-actions github-actions Bot added Curriculum material Curriculum materials, lessons for students, etc stage1 labels Sep 11, 2026
@github-actions

Copy link
Copy Markdown

🌐 Preview URL: https://pr-272-frcsoftware.frcsoftware.workers.dev

Signed-off-by: samfreund <samf.236@proton.me>
Signed-off-by: samfreund <samf.236@proton.me>
Signed-off-by: samfreund <samf.236@proton.me>
…mple code

The Stage 1A course content referenced NiDsXboxController while the actual
Stage 1A solution code uses XboxController. Standardize on XboxController/
CommandXboxController throughout.
@samfreund
samfreund requested a review from Adrianamm September 11, 2026 04:02
@samfreund
samfreund marked this pull request as ready for review September 11, 2026 04:02
@Spaceman113138

Copy link
Copy Markdown
Contributor

1b will need updated sim-gui files for the keyboard control to work properly with XboxController.

…1 buttons)

The stage1b solution code uses xbox.leftBumper() (button 9),
xbox.rightBumper() (button 10), and xbox.a() (button 0) on
CommandXboxController. The previous simgui only had 6 buttons,
which didn't support the bumper buttons. Updated to 11 buttons
with the same key configuration as stage1a.
@Adrianamm
Adrianamm merged commit 29fc5d8 into frcsoftware:main Sep 11, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Curriculum material Curriculum materials, lessons for students, etc examples stage0 stage1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants