Skip to content

Wire MetalFX temporal upscaling into Renderer#143

Merged
stainlu merged 1 commit intomainfrom
worktree-agent-a8ab615a
Apr 6, 2026
Merged

Wire MetalFX temporal upscaling into Renderer#143
stainlu merged 1 commit intomainfrom
worktree-agent-a8ab615a

Conversation

@stainlu
Copy link
Copy Markdown
Owner

@stainlu stainlu commented Apr 6, 2026

Summary

  • Add metalfx_enabled field and enable_metalfx/is_metalfx_enabled methods to the generic Renderer<D> (no-op on wgpu, active on Metal)
  • Add cfg-gated impl Renderer<MetalDevice> with setup_metalfx_upscaling and resize_metalfx that validate MetalFX upscaler creation
  • Add supports_metalfx capability query on GpuContext<MetalDevice>
  • Document the full integration path (50% PBR render -> MetalFX upscale -> post-process ping buffer) for follow-up work

Test plan

  • cargo check --workspace (wgpu path)
  • cargo check -p euca-render --features metal-native (Metal path)
  • cargo test --workspace (all pass)
  • cargo clippy --workspace -- -D warnings (clean)
  • cargo fmt --all -- --check (clean)

🤖 Generated with Claude Code

Add optional MetalFX support to the generic Renderer<D>:
- `metalfx_enabled` field (stored on all backends, no-op on wgpu)
- `enable_metalfx` / `is_metalfx_enabled` methods on generic impl
- `setup_metalfx_upscaling` / `resize_metalfx` on Metal-native impl
  (cfg-gated behind target_os = "macos" + feature = "metal-native")
- `supports_metalfx` capability query on GpuContext<MetalDevice>

The Metal-native impl validates upscaler creation and documents the
full integration path (render PBR at 50% res, upscale, feed into
post-process ping buffer) for a follow-up.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying euca-engine with  Cloudflare Pages  Cloudflare Pages

Latest commit: c68eb96
Status: ✅  Deploy successful!
Preview URL: https://1ac4ffa8.euca-engine.pages.dev
Branch Preview URL: https://worktree-agent-a8ab615a.euca-engine.pages.dev

View logs

@stainlu stainlu merged commit 552283e into main Apr 6, 2026
6 of 8 checks passed
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