Skip to content

feat: Add demo scenes for testing 3D raycast collider annotations#1665

Merged
hatayama merged 1 commit into
v3-betafrom
feat/raycast-demo-scenes
Jul 9, 2026
Merged

feat: Add demo scenes for testing 3D raycast collider annotations#1665
hatayama merged 1 commit into
v3-betafrom
feat/raycast-demo-scenes

Conversation

@hatayama

@hatayama hatayama commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds two self-contained demo scenes for exercising screenshot --annotate-raycast-grid and the raycast tool: a flat 4x4 tile collider grid, and a perspective-camera scene with split/rotated-L collider shapes.
  • Both scenes include a UI blocker overlay so the UI-occlusion handling added in C2/C3 (covered PhysicsCollider entries get omitted from AnnotatedElements) has something concrete to verify against.

User Impact

  • Before: there was no ready-made scene to try the raycast grid annotation feature (dense sampling, layer summaries, L-shaped/split collider bounds, UI occlusion) without building a test scene by hand.
  • After: opening either demo scene and running screenshot --capture-mode rendering --annotate-raycast-grid true gives immediate, reproducible output to inspect.

Changes

  • Assets/Scenes/RaycastAnnotationDemoScene.unity (+ .meta): flat tile grid (Tile_R0_C0..R3_C3), Main Camera, Directional Light, EventSystem, and a UI blocker Canvas.
  • Assets/Scenes/RaycastAnnotationPerspectiveDemoScene.unity (+ .meta): perspective split-grid and rotated-L collider layouts with the same UI blocker setup.
  • Ported unchanged from main's a2abaf04 (fix: Raycast annotations show collider coverage without covered UI targets #1500); both scenes reference only built-in Unity/URP assets with no custom scripts or external prefabs, so no other file changes were needed.
  • Not registered in ProjectSettings/EditorBuildSettings.asset — main doesn't register them either, matching the existing pattern for this repo's other manually-opened demo scenes (SimulateKeyboardDemoScene, SimulateMouseDemoScene).

Verification

  • uloop compile → 0 errors, 0 warnings
  • Confirmed both scenes' .meta GUIDs don't collide with any existing v3-beta asset
  • Confirmed every guid: reference inside both scene files resolves to a built-in Unity/URP asset already present in this project (no custom scripts, prefabs, or materials referenced)

Review in cubic

These two scenes give agents and developers a self-contained scene to
exercise screenshot --annotate-raycast-grid and the raycast tool
against: a flat tile grid and a perspective split/rotated-L collider
layout, both with a UI blocker overlay to verify UI-occlusion
handling. Both scenes only reference built-in Unity/URP assets with
no custom scripts or external prefabs, so they copy over unchanged.
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (4)
  • Assets/Scenes/RaycastAnnotationDemoScene.unity is excluded by !**/*.unity and included by none
  • Assets/Scenes/RaycastAnnotationDemoScene.unity.meta is excluded by none and included by none
  • Assets/Scenes/RaycastAnnotationPerspectiveDemoScene.unity is excluded by !**/*.unity and included by none
  • Assets/Scenes/RaycastAnnotationPerspectiveDemoScene.unity.meta is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 33c6f22a-5bec-4f8d-9bd5-646ea60408ec

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/raycast-demo-scenes

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.

@hatayama
hatayama merged commit ae13a21 into v3-beta Jul 9, 2026
6 checks passed
@hatayama
hatayama deleted the feat/raycast-demo-scenes branch July 9, 2026 13:29
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