Skip to content

[Crazy RoboTaxi] Improve Crazy Robotaxi controller handling - #613

Open
aidanfnv wants to merge 1 commit into
dev/aidanf/game/crazy-robotaxi/controlsfrom
dev/aidanf/game/crazy-robotaxi/controller-input
Open

aidanfnv wants to merge 1 commit into
dev/aidanf/game/crazy-robotaxi/controlsfrom
dev/aidanf/game/crazy-robotaxi/controller-input

Conversation

@aidanfnv

Copy link
Copy Markdown
Collaborator

Split off from #553, this change modifies the controller handling to make the game feel better on a controller, such as by restoring analog steering. Previously, to make steering more responsive, it was apparently changed to be digital.

@aidanfnv
aidanfnv added this pull request to stack #610 September 11, 2026 00:21
@copy-pr-bot

copy-pr-bot Bot commented Sep 11, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@greptile-apps

greptile-apps Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge, with no outstanding actionable findings identified.

Summary

This PR improves Crazy Robotaxi controller handling and input-source arbitration.

  • Restores analog gamepad steering and adds rescaled stick and trigger dead zones.
  • Aligns keyboard commands with the shared arcade-driving command shape.
  • Smooths direct steering using configured steering and return rates.
  • Lets deliberate keyboard input reclaim control without neutral controller events immediately overriding it.
  • Adds focused control, gameplay, and input-source tests.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
    K[Keyboard events] --> DI[DriverInput]
    G[Gamepad events] --> DZ[Dead-zone rescaling]
    DZ --> DI
    W[Wheel events] --> DI
    DI --> ARB[Most recent deliberate input arbitration]
    ARB --> DC[DriverCommand]
    DC --> SD[Rate-limited direct steering]
    SD --> VS[Updated vehicle state]
Loading

Reviews (8) · Last reviewed commit: "Improve Crazy Robotaxi controller handli..."

@aidanfnv
aidanfnv force-pushed the dev/aidanf/game/crazy-robotaxi/controller-input branch from 7627d57 to 54114a8 Compare September 11, 2026 02:30
@aidanfnv
aidanfnv force-pushed the dev/aidanf/game/crazy-robotaxi/controller-input branch 2 times, most recently from 1827096 to edcfb83 Compare September 12, 2026 00:14
@aidanfnv

Copy link
Copy Markdown
Collaborator Author

/ok to test edcfb83

@aidanfnv
aidanfnv force-pushed the dev/aidanf/game/crazy-robotaxi/controller-input branch 2 times, most recently from c9a52c5 to 6a75f82 Compare September 12, 2026 01:29
@aidanfnv
aidanfnv force-pushed the dev/aidanf/game/crazy-robotaxi/controller-input branch from 6a75f82 to fabbef9 Compare September 14, 2026 16:40
@aidanfnv

Copy link
Copy Markdown
Collaborator Author

/ok to test fabbef9

Signed-off-by: Aidan Foster <aidanf@nvidia.com>
@aidanfnv
aidanfnv force-pushed the dev/aidanf/game/crazy-robotaxi/controller-input branch from fabbef9 to b629dd1 Compare September 14, 2026 22:52
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