Skip to content

add render mode that doesn't intersect rays for debugging purposes#336

Merged
jeffamstutz merged 1 commit intoKhronosGroup:next_releasefrom
jeffamstutz:helide_test_frame
Jan 9, 2026
Merged

add render mode that doesn't intersect rays for debugging purposes#336
jeffamstutz merged 1 commit intoKhronosGroup:next_releasefrom
jeffamstutz:helide_test_frame

Conversation

@jeffamstutz
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings January 9, 2026 22:15
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new TEST_FRAME render mode for debugging purposes that visualizes ray directions without performing ray-scene intersections. This allows developers to verify camera setup and ray generation without the overhead of scene traversal.

  • Adds TEST_FRAME enum value to RenderMode
  • Implements early-exit rendering logic that maps ray directions to RGB colors
  • Updates JSON configuration to include the new mode

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/devices/helide/renderer/Renderer.h Adds TEST_FRAME enum value to RenderMode
src/devices/helide/renderer/Renderer.cpp Implements string-to-enum conversion and rendering logic for TEST_FRAME mode
src/devices/helide/HelideDefinitions.json Adds "testFrame" to allowed renderer mode values
src/devices/helide/frame/Frame.cpp Minor formatting improvement to getProperty function signature

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jeffamstutz jeffamstutz merged commit ad24faf into KhronosGroup:next_release Jan 9, 2026
14 checks passed
@jeffamstutz jeffamstutz deleted the helide_test_frame branch January 13, 2026 14:48
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