From 4fe647e13e3483e7fe01e6466c3871a20892963f Mon Sep 17 00:00:00 2001 From: skal Date: Tue, 3 Feb 2026 19:47:52 +0100 Subject: fix: Implement proper skybox rendering with Perlin noise - Added ObjectType::SKYBOX for dedicated skybox rendering. - Created assets/final/shaders/skybox.wgsl for background rendering. - Implemented a two-pass rendering strategy in Renderer3D::render: - First pass renders the skybox without depth writes. - Second pass renders scene objects with depth testing. - Corrected GlobalUniforms struct in common_uniforms.wgsl and src/3d/renderer.h to include and explicit padding for 112-byte alignment. - Updated Renderer3D::update_uniforms to set the new and zero-initialize padding. - Reverted sky sampling logic in renderer_3d.wgsl to for SDF misses, preventing background bleed-through. - Updated test_3d_render.cc to include a SKYBOX object with Perlin noise. handoff(Gemini): The skybox is now correctly rendered with Perlin noise as a dedicated background pass. Objects render correctly without transparency to the sky. All necessary C++ and WGSL shader changes are implemented and verified. --- src/3d/renderer.h | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'src/3d/renderer.h') diff --git a/src/3d/renderer.h b/src/3d/renderer.h index 3caa329..148a521 100644 --- a/src/3d/renderer.h +++ b/src/3d/renderer.h @@ -18,6 +18,8 @@ struct GlobalUniforms { mat4 view_proj; vec4 camera_pos_time; // xyz = camera_pos, w = time vec4 params; // x = num_objects, yzw = padding + vec2 resolution; + vec2 padding; }; // Matches the GPU struct layout @@ -64,6 +66,7 @@ class Renderer3D { private: void create_pipeline(); + void create_skybox_pipeline(); void create_default_resources(); void update_uniforms(const Scene& scene, const Camera& camera, float time); @@ -73,6 +76,8 @@ class Renderer3D { WGPURenderPipeline pipeline_ = nullptr; WGPUBindGroup bind_group_ = nullptr; + WGPURenderPipeline skybox_pipeline_ = nullptr; + WGPUBindGroup skybox_bind_group_ = nullptr; WGPUBuffer global_uniform_buffer_ = nullptr; WGPUBuffer object_storage_buffer_ = nullptr; -- cgit v1.2.3