Skip to content

[Crazy RoboTaxi] Assign spawns to race courses, add first frame thumbnails to map/course selection screens - #609

Open
aidanfnv wants to merge 5 commits into
dev/aidanf/game/crazy-robotaxi/map-format-cleanupfrom
dev/aidanf/game/crazy-robotaxi/map-course-previews
Open

aidanfnv wants to merge 5 commits into
dev/aidanf/game/crazy-robotaxi/map-format-cleanupfrom
dev/aidanf/game/crazy-robotaxi/map-course-previews

Conversation

@aidanfnv

Copy link
Copy Markdown
Collaborator

Split off from #553, this change makes it so that each race course in a map has a spawn associated with them instead of using the map's default spawn, and adds each course's spawn point's first frame to the course selection screen as a thumbnail preview. It does the same with the map selection screen using the default spawn's first frame.

@copy-pr-bot

copy-pr-bot Bot commented Sep 10, 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 10, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge, with no outstanding previous findings or accepted new issues.

Summary

This PR assigns an authored spawn to each race course and uses spawn imagery for map and course selection previews.

  • Extends map schema, headers, runtime types, serialization, and documentation with course spawn metadata and optional map thumbnails.
  • Compiles and caches spawn-specific scenes, then selects the course’s spawn when starting a race.
  • Adds thumbnail-backed map and course grids with responsive sizing and horizontal scrolling.
  • Updates bundled maps and tests for schema validation, scene selection, discovery, and UI layout.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Robotaxi map YAML] --> B[Load map header]
    B --> C[Map menu thumbnail]
    B --> D[Race-course metadata]
    D --> E[Course spawn ID]
    D --> F[Spawn image thumbnail]
    C --> G[Map selection grid]
    F --> H[Course selection grid]
    H --> I[GameSelection]
    I --> J[SceneRequest with spawn ID]
    J --> K[Compile spawn-specific archive]
    K --> L[Load race scene at course spawn]
Loading

Reviews (7) · Last reviewed commit: "[crazy-robotaxi] Fix UI test import orde..."

Comment thread apps/crazy_robotaxi/crazy_robotaxi/ui.py Outdated
Comment thread apps/crazy_robotaxi/crazy_robotaxi/ui.py Outdated
@aidanfnv
aidanfnv added this pull request to stack #610 September 10, 2026 23:17
@aidanfnv

Copy link
Copy Markdown
Collaborator Author

/ok to test dfa64d9

@aidanfnv

Copy link
Copy Markdown
Collaborator Author

/ok to test cf49060

@aidanfnv
aidanfnv force-pushed the dev/aidanf/game/crazy-robotaxi/map-course-previews branch from cf49060 to aba1b9e Compare September 12, 2026 00:53
@aidanfnv
aidanfnv force-pushed the dev/aidanf/game/crazy-robotaxi/map-course-previews branch from aba1b9e to 2332758 Compare September 12, 2026 01:30
@aidanfnv
aidanfnv force-pushed the dev/aidanf/game/crazy-robotaxi/map-course-previews branch from 2332758 to 232bacb Compare September 14, 2026 16:40
@aidanfnv

Copy link
Copy Markdown
Collaborator Author

/ok to test 232bacb

@aidanfnv
aidanfnv force-pushed the dev/aidanf/game/crazy-robotaxi/map-course-previews branch from 232bacb to cc6ddd0 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