Feature/optix renderer by pmudry · Pull Request #15 · pmudry/RayON

pmudry · 2026-03-15T00:37:26Z

No description provided.

New renderer (mode 4) using NVIDIA OptiX SDK for hardware-accelerated ray intersection. Includes custom intersection programs for spheres and rectangles, optimized device programs (direct RNG, rsqrtf normalize, manual Schlick), persistent GPU buffers, and SDF geometry filtering. Benchmarks: ~2.76s vs CUDA ~2.50s at 1024spp/1080p. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

github-actions · 2026-03-15T00:39:35Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

Copilot

Pull request overview

Adds an optional NVIDIA OptiX rendering backend (offline + SDL progressive) to RayON, integrating it into the CLI/menu and build system alongside existing CPU/CUDA renderers.

Changes:

Introduces OptiX offline renderer (RendererOptiX) and OptiX+SDL progressive renderer (RendererOptiXProgressive).
Adds OptiX host/device implementation and PTX build step (custom NVCC PTX compilation + SDK detection).
Updates ancillary docs/scripts/config (README known-issues text, benchmark script, .clangd, new CLAUDE.md, new performance note doc).

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 11 comments.

Show a summary per file

File	Description
`src/rayon/main.cc`	Adds OptiX modes (4/5) to CLI validation, menu, and renderer switch.
`src/rayon/gpu_renderers/renderer_optix_host.hpp`	New offline OptiX `IRenderer` implementation calling OptiX host API.
`src/rayon/gpu_renderers/renderer_optix_progressive_host.hpp`	New SDL progressive OptiX renderer mirroring CUDA progressive flow.
`src/rayon/gpu_renderers/optix/optix_renderer.cu`	Host-side OptiX state/pipeline/GAS/SBT setup, launches, downloads, cleanup.
`src/rayon/gpu_renderers/optix/optix_programs.cu`	OptiX device programs (raygen/miss/closesthit/intersection) compiled to PTX.
`src/rayon/gpu_renderers/optix/optix_params.h`	Shared OptiX host/device structs/enums (launch params, SBT records, PRD).
`CMakeLists.txt`	Adds OptiX source + optional SDK discovery and PTX build target.
`README.md`	Edits “Known issues” text.
`explanations/VULKAN_VS_CUDA_PERFORMANCE.md`	Adds a new explanation note comparing HW RT vs CUDA path tracing.
`CLAUDE.md`	Adds repository guidance (build commands, layout, conventions).
`bench.sh`	Adds optional mode parameter to benchmark different backends.
`.clangd`	Adds OptiX include path.

You can also share your feedback on Copilot code review. Take the survey.

src/rayon/main.cc

CMakeLists.txt

src/rayon/gpu_renderers/optix/optix_renderer.cu

src/rayon/gpu_renderers/renderer_optix_progressive_host.hpp

src/rayon/gpu_renderers/renderer_optix_host.hpp

src/rayon/gpu_renderers/optix/optix_renderer.cu

README.md

CLAUDE.md

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

github-actions · 2026-03-15T00:46:26Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

github-actions · 2026-03-15T00:46:43Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

github-actions · 2026-03-15T00:46:54Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

github-actions · 2026-03-15T00:47:18Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

github-actions · 2026-03-15T00:47:29Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

github-actions · 2026-03-15T00:47:48Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

github-actions · 2026-03-15T00:47:55Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

github-actions · 2026-03-15T00:48:01Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

github-actions · 2026-03-15T00:48:15Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

github-actions · 2026-03-15T00:48:25Z

Docs out of sync?

This PR changes 7 source file(s) but no pages under website/docs/ were updated.

If this is a significant change (new feature, new CLI flag, new material/geometry type, performance improvement), please update the relevant documentation page. See the mapping in .github/copilot-instructions.md.

If this change genuinely doesn't affect the docs, you can ignore this message.

pmudry · 2026-03-15T00:49:43Z

Merging...

pmudry

Done

pmudry and others added 5 commits March 10, 2026 17:50

Some infos for claude

018663f

Update README.md

ffb07e2

First OptiX integration

ce76185

Merge branch 'main' into feature/optix-renderer

4180835

pmudry requested a review from Copilot March 15, 2026 00:39

Copilot started reviewing on behalf of pmudry March 15, 2026 00:40 View session

Copilot AI reviewed Mar 15, 2026

View reviewed changes

Potential fix for pull request finding

4ff6e2b

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

pmudry and others added 2 commits March 15, 2026 01:46

Potential fix for pull request finding

dd6d96a

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

66dcba9

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

pmudry and others added 2 commits March 15, 2026 01:47

Potential fix for pull request finding

cdd3100

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

df6b33e

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

pmudry and others added 2 commits March 15, 2026 01:47

Potential fix for pull request finding

ccb9d8f

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

b41cfaf

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

fd2efd4

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

pmudry and others added 2 commits March 15, 2026 01:48

Potential fix for pull request finding

f56105d

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

ecf7c92

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

pmudry commented Mar 15, 2026

View reviewed changes

pmudry merged commit 70c0ee7 into main Mar 15, 2026
1 check passed

pmudry deleted the feature/optix-renderer branch March 15, 2026 20:05

Conversation

pmudry commented Mar 15, 2026

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

github-actions bot commented Mar 15, 2026

Docs out of sync?

Uh oh!

pmudry commented Mar 15, 2026

Uh oh!

pmudry left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants