Various fixes and cleanup for post-process pass (#551)

adriengivry · web-flow · commit bb1a6138462e · 2025-05-22T20:15:35.000-04:00
diff --git a/Resources/Engine/Shaders/PostProcess/Bloom.ovfx b/Resources/Engine/Shaders/PostProcess/Bloom.ovfx
@@ -16,7 +16,7 @@ void main()
 #version 450 core
 
 in vec2 TexCoords;
-out vec3 FRAGMENT_COLOR;
+out vec4 FRAGMENT_COLOR;
 
 uniform sampler2D _InputTexture;
 uniform sampler2D _BloomTexture;
@@ -26,5 +26,5 @@ void main()
 {
     const vec3 hdrColor = texture(_InputTexture, TexCoords).rgb;
     const vec3 bloomColor = texture(_BloomTexture, TexCoords).rgb;
-    FRAGMENT_COLOR = mix(hdrColor, bloomColor, _BloomStrength);
+    FRAGMENT_COLOR = vec4(mix(hdrColor, bloomColor, _BloomStrength), 1.0);
 }
diff --git a/Resources/Engine/Shaders/PostProcess/BloomDownsampling.ovfx b/Resources/Engine/Shaders/PostProcess/BloomDownsampling.ovfx
@@ -18,7 +18,7 @@ void main()
 #version 450 core
 
 in vec2 TexCoords;
-out vec3 downsample;
+out vec4 FRAGMENT_COLOR;
 
 uniform sampler2D _InputTexture;
 uniform vec2 _InputResolution;
@@ -88,6 +88,8 @@ void main()
     // to effectively yield this sum. We get:
     // 0.125*5 + 0.03125*4 + 0.0625*4 = 1
 
+    vec3 downsample = vec3(0.0);
+
     // Check if we need to perform Karis average on each block of 4 samples
 #if defined(KARIS_AVERAGE)
     vec3 groups[5];
@@ -112,4 +114,6 @@ void main()
     downsample += (b+d+f+h)*0.0625;      // ok
     downsample += (j+k+l+m)*0.125;       // ok
 #endif
+
+    FRAGMENT_COLOR = vec4(downsample, 1.0);
 }
diff --git a/Resources/Engine/Shaders/PostProcess/BloomUpsampling.ovfx b/Resources/Engine/Shaders/PostProcess/BloomUpsampling.ovfx
@@ -16,7 +16,7 @@ void main()
 #version 450 core
 
 in vec2 TexCoords;
-out vec3 upsample;
+out vec4 FRAGMENT_COLOR;
 
 uniform sampler2D _InputTexture;
 uniform float _FilterRadius;
@@ -47,8 +47,10 @@ void main()
     //  1   | 1 2 1 |
     // -- * | 2 4 2 |
     // 16   | 1 2 1 |
-    upsample = e*4.0;
+    vec3 upsample = e*4.0;
     upsample += (b+d+f+h)*2.0;
     upsample += (a+c+g+i);
     upsample *= 1.0 / 16.0;
+
+    FRAGMENT_COLOR = vec4(upsample, 1.0);
 }
diff --git a/Sources/Overload/OvCore/include/OvCore/Rendering/PostProcess/AutoExposureEffect.h b/Sources/Overload/OvCore/include/OvCore/Rendering/PostProcess/AutoExposureEffect.h
@@ -8,6 +8,7 @@
 
 #include <chrono>
 
+#include <OvCore/Rendering/PingPongFramebuffer.h>
 #include <OvCore/Rendering/PostProcess/AEffect.h>
 #include <OvRendering/Data/Material.h>
 
@@ -57,8 +58,7 @@ namespace OvCore::Rendering::PostProcess
 	private:
 		std::optional<std::chrono::high_resolution_clock::time_point> m_previousTime;
 		OvRendering::HAL::Framebuffer m_luminanceBuffer;
-		std::array<OvRendering::HAL::Framebuffer, 2> m_exposurePingPongBuffer;
-		uint8_t m_exposurePingPongIndex = 0;
+		PingPongFramebuffer m_exposurePingPongBuffer;
 		OvRendering::Data::Material m_luminanceMaterial;
 		OvRendering::Data::Material m_exposureMaterial;
 		OvRendering::Data::Material m_compensationMaterial;
diff --git a/Sources/Overload/OvCore/include/OvCore/Rendering/PostProcess/BloomEffect.h b/Sources/Overload/OvCore/include/OvCore/Rendering/PostProcess/BloomEffect.h
@@ -6,7 +6,6 @@
 
 #pragma once
 
-#include <OvCore/Rendering/PingPongFramebuffer.h>
 #include <OvCore/Rendering/PostProcess/AEffect.h>
 #include <OvRendering/Data/Material.h>
 #include <OvRendering/HAL/Framebuffer.h>
diff --git a/Sources/Overload/OvCore/include/OvCore/Rendering/PostProcessRenderPass.h b/Sources/Overload/OvCore/include/OvCore/Rendering/PostProcessRenderPass.h
@@ -8,6 +8,7 @@
 
 #include <OvRendering/Core/ARenderPass.h>
 #include <OvRendering/HAL/Framebuffer.h>
+#include <OvCore/Rendering/PingPongFramebuffer.h>
 #include <OvCore/Rendering/PostProcess/AEffect.h>
 
 namespace OvCore::Rendering
@@ -30,6 +31,6 @@ namespace OvCore::Rendering
 	private:
 		OvRendering::Data::Material m_blitMaterial;
 		std::vector<std::unique_ptr<PostProcess::AEffect>> m_effects;
-		std::array<OvRendering::HAL::Framebuffer, 2> m_pingPongBuffers;
+		PingPongFramebuffer m_pingPongBuffers;
 	};
-}
+}
diff --git a/Sources/Overload/OvCore/src/OvCore/ECS/Components/CPostProcessStack.cpp b/Sources/Overload/OvCore/src/OvCore/ECS/Components/CPostProcessStack.cpp
@@ -81,25 +81,25 @@ void OvCore::ECS::Components::CPostProcessStack::OnInspector(OvUI::Internal::Wid
 	auto& fxaaSettings = m_settings.Get<Rendering::PostProcess::FXAAEffect, Rendering::PostProcess::FXAASettings>();
 	auto& tonemappingSettings = m_settings.Get<Rendering::PostProcess::TonemappingEffect, Rendering::PostProcess::TonemappingSettings>();
 
-	OvCore::Helpers::GUIDrawer::DrawBoolean(p_root, "Bloom Enabled", bloomSettings.enabled);
-	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Bloom Intensity", bloomSettings.intensity, 0.1f, 0.0f, Rendering::PostProcess::BloomConstants::kMaxBloomIntensity);
-	OvCore::Helpers::GUIDrawer::DrawScalar<int>(p_root, "Bloom Passes", bloomSettings.passes, 1, Rendering::PostProcess::BloomConstants::kMinPassCount, Rendering::PostProcess::BloomConstants::kMaxPassCount);
-	
-	p_root.CreateWidget<OvUI::Widgets::Visual::Separator>();
-	p_root.CreateWidget<OvUI::Widgets::Layout::Spacing>();
-
 	OvCore::Helpers::GUIDrawer::DrawBoolean(p_root, "Auto Exposure Enabled", autoExposureSettings.enabled);
 	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Auto Exposure Center Weight Bias", autoExposureSettings.centerWeightBias, 0.1f, 0.0f, 1.0f);
-	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Auto Exposure Min Luminance (EV)", autoExposureSettings.minLuminanceEV, 1.0f, OvCore::Helpers::GUIDrawer::_MIN_FLOAT, OvCore::Helpers::GUIDrawer::_MAX_FLOAT);
-	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Auto Exposure Max Luminance (EV)", autoExposureSettings.maxLuminanceEV, 1.0f, OvCore::Helpers::GUIDrawer::_MIN_FLOAT, OvCore::Helpers::GUIDrawer::_MAX_FLOAT);
-	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Auto Exposure Exposure Compensation (EV)", autoExposureSettings.exposureCompensationEV, 1.0f, OvCore::Helpers::GUIDrawer::_MIN_FLOAT, OvCore::Helpers::GUIDrawer::_MAX_FLOAT);
+	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Auto Exposure Min Luminance (EV)", autoExposureSettings.minLuminanceEV, 0.1f, OvCore::Helpers::GUIDrawer::_MIN_FLOAT, OvCore::Helpers::GUIDrawer::_MAX_FLOAT);
+	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Auto Exposure Max Luminance (EV)", autoExposureSettings.maxLuminanceEV, 0.1f, OvCore::Helpers::GUIDrawer::_MIN_FLOAT, OvCore::Helpers::GUIDrawer::_MAX_FLOAT);
+	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Auto Exposure Exposure Compensation (EV)", autoExposureSettings.exposureCompensationEV, 0.1f, OvCore::Helpers::GUIDrawer::_MIN_FLOAT, OvCore::Helpers::GUIDrawer::_MAX_FLOAT);
 	OvCore::Helpers::GUIDrawer::DrawBoolean(p_root, "Auto Exposure Progressive", autoExposureSettings.progressive);
 	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Auto Exposure Speed Up", autoExposureSettings.speedUp);
 	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Auto Exposure Speed Down", autoExposureSettings.speedDown);
 
 	p_root.CreateWidget<OvUI::Widgets::Visual::Separator>();
 	p_root.CreateWidget<OvUI::Widgets::Layout::Spacing>();
 
+	OvCore::Helpers::GUIDrawer::DrawBoolean(p_root, "Bloom Enabled", bloomSettings.enabled);
+	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Bloom Intensity", bloomSettings.intensity, 0.1f, 0.0f, Rendering::PostProcess::BloomConstants::kMaxBloomIntensity);
+	OvCore::Helpers::GUIDrawer::DrawScalar<int>(p_root, "Bloom Passes", bloomSettings.passes, 1, Rendering::PostProcess::BloomConstants::kMinPassCount, Rendering::PostProcess::BloomConstants::kMaxPassCount);
+
+	p_root.CreateWidget<OvUI::Widgets::Visual::Separator>();
+	p_root.CreateWidget<OvUI::Widgets::Layout::Spacing>();
+
 	OvCore::Helpers::GUIDrawer::DrawBoolean(p_root, "Tonemapping Enabled", tonemappingSettings.enabled);
 	OvCore::Helpers::GUIDrawer::DrawScalar<float>(p_root, "Tonemapping Exposure", tonemappingSettings.exposure, 0.05f, 0.0f);
 	Helpers::GUIDrawer::CreateTitle(p_root, "Tonemapping Mode");
@@ -112,8 +112,7 @@ void OvCore::ECS::Components::CPostProcessStack::OnInspector(OvUI::Internal::Wid
 		{ 4, "Uncharted 2 (Filmic)" },
 		{ 5, "ACES" }
 	};
-	tonemappingMode.ValueChangedEvent += [this](int p_choice)
-	{
+	tonemappingMode.ValueChangedEvent += [this](int p_choice) {
 		auto& tonemappingSettings = m_settings.Get<Rendering::PostProcess::TonemappingEffect, Rendering::PostProcess::TonemappingSettings>();
 		tonemappingSettings.mode = static_cast<OvCore::Rendering::PostProcess::ETonemappingMode>(p_choice);
 	};
diff --git a/Sources/Overload/OvCore/src/OvCore/Rendering/PostProcess/AutoExposureEffect.cpp b/Sources/Overload/OvCore/src/OvCore/Rendering/PostProcess/AutoExposureEffect.cpp
@@ -18,12 +18,9 @@ OvCore::Rendering::PostProcess::AutoExposureEffect::AutoExposureEffect(
 	OvRendering::Core::CompositeRenderer& p_renderer
 ) :	AEffect(p_renderer),
 	m_luminanceBuffer{ "Luminance" },
-	m_exposurePingPongBuffer{
-		OvRendering::HAL::Framebuffer{"ExposurePingPong0"},
-		OvRendering::HAL::Framebuffer{"ExposurePingPong1"}
-	}
+	m_exposurePingPongBuffer{ "Exposure" }
 {
-	for (auto& buffer : m_exposurePingPongBuffer)
+	for (auto& buffer : m_exposurePingPongBuffer.GetFramebuffers())
 	{
 		FramebufferUtil::SetupFramebuffer(
 			buffer,
@@ -76,9 +73,9 @@ void OvCore::Rendering::PostProcess::AutoExposureEffect::Draw(
 	}
 	m_previousTime = currentTime;
 
-	auto& previousExposure = m_exposurePingPongBuffer[(m_exposurePingPongIndex + 1) % 2];
-	auto& currentExposure = m_exposurePingPongBuffer[m_exposurePingPongIndex];
-	m_exposurePingPongIndex = (m_exposurePingPongIndex + 1) % 2;
+	auto& previousExposure = m_exposurePingPongBuffer[0];
+	auto& currentExposure = m_exposurePingPongBuffer[1];
+	++m_exposurePingPongBuffer;
 
 	// Exposure adaptation
 	m_exposureMaterial.SetProperty("_LuminanceTexture", &luminanceTex.value(), true);
diff --git a/Sources/Overload/OvCore/src/OvCore/Rendering/PostProcessRenderPass.cpp b/Sources/Overload/OvCore/src/OvCore/Rendering/PostProcessRenderPass.cpp
@@ -18,12 +18,9 @@
 
 OvCore::Rendering::PostProcessRenderPass::PostProcessRenderPass(OvRendering::Core::CompositeRenderer& p_renderer) :
 	OvRendering::Core::ARenderPass(p_renderer),
-	m_pingPongBuffers{
-		OvRendering::HAL::Framebuffer{"PostProcessBlitPingPong0"},
-		OvRendering::HAL::Framebuffer{"PostProcessBlitPingPong1"}
-	}
+	m_pingPongBuffers{ "PostProcessBlit" }
 {
-	for (auto& buffer : m_pingPongBuffers)
+	for (auto& buffer : m_pingPongBuffers.GetFramebuffers())
 	{
 		OvCore::Rendering::FramebufferUtil::SetupFramebuffer(
 			buffer, 1, 1, false, false, false
@@ -65,12 +62,8 @@ void OvCore::Rendering::PostProcessRenderPass::Draw(OvRendering::Data::PipelineS
 
 	if (auto stack = FindPostProcessStack(scene))
 	{
-		uint32_t passIndex = 0;
-
 		auto& framebuffer = m_renderer.GetFrameDescriptor().outputBuffer.value();
 
-		const uint64_t kPingPongBufferSize = m_pingPongBuffers.size();
-		
 		m_renderer.Blit(p_pso, framebuffer, m_pingPongBuffers[0], m_blitMaterial);
 
 		for (auto& effect : m_effects)
@@ -81,17 +74,15 @@ void OvCore::Rendering::PostProcessRenderPass::Draw(OvRendering::Data::PipelineS
 			{
 				effect->Draw(
 					p_pso,
-					m_pingPongBuffers[passIndex % kPingPongBufferSize],
-					m_pingPongBuffers[(passIndex + 1) % kPingPongBufferSize],
+					m_pingPongBuffers[0],
+					m_pingPongBuffers[1],
 					settings
 				);
 
-				++passIndex;
+				++m_pingPongBuffers;
 			}
 		}
 
-		const uint64_t lastIndex = passIndex % kPingPongBufferSize;
-
-		m_renderer.Blit(p_pso, m_pingPongBuffers[lastIndex], framebuffer, m_blitMaterial);
+		m_renderer.Blit(p_pso, m_pingPongBuffers[0], framebuffer, m_blitMaterial);
 	}
 }
diff --git a/Sources/Overload/OvRendering/src/OvRendering/Context/Driver.cpp b/Sources/Overload/OvRendering/src/OvRendering/Context/Driver.cpp
@@ -51,6 +51,10 @@ OvRendering::Context::Driver::~Driver()
 void OvRendering::Context::Driver::OnFrameCompleted()
 {
 	m_gfxBackend->OnFrameCompleted();
+
+	// Prevents state leak between frames, and especially useful when external code (like ImGui)
+	// requires a "neutral" pipeline state.
+	ResetPipelineState();
 }
 
 void OvRendering::Context::Driver::SetViewport(uint32_t p_x, uint32_t p_y, uint32_t p_width, uint32_t p_height)

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ void main()`
`16`	`16`	`#version 450 core`
`17`	`17`
`18`	`18`	`in vec2 TexCoords;`
`19`		`-out vec3 FRAGMENT_COLOR;`
	`19`	`+out vec4 FRAGMENT_COLOR;`
`20`	`20`
`21`	`21`	`uniform sampler2D _InputTexture;`
`22`	`22`	`uniform sampler2D _BloomTexture;`
`@@ -26,5 +26,5 @@ void main()`
`26`	`26`	`{`
`27`	`27`	`const vec3 hdrColor = texture(_InputTexture, TexCoords).rgb;`
`28`	`28`	`const vec3 bloomColor = texture(_BloomTexture, TexCoords).rgb;`
`29`		`- FRAGMENT_COLOR = mix(hdrColor, bloomColor, _BloomStrength);`
	`29`	`+ FRAGMENT_COLOR = vec4(mix(hdrColor, bloomColor, _BloomStrength), 1.0);`
`30`	`30`	`}`
Original file line number	Diff line number	Diff line change
`@@ -18,12 +18,9 @@`
`18`	`18`
`19`	`19`	`OvCore::Rendering::PostProcessRenderPass::PostProcessRenderPass(OvRendering::Core::CompositeRenderer& p_renderer) :`
`20`	`20`	`OvRendering::Core::ARenderPass(p_renderer),`
`21`		`- m_pingPongBuffers{`
`22`		`- OvRendering::HAL::Framebuffer{"PostProcessBlitPingPong0"},`
`23`		`- OvRendering::HAL::Framebuffer{"PostProcessBlitPingPong1"}`
`24`		`- }`
	`21`	`+ m_pingPongBuffers{ "PostProcessBlit" }`
`25`	`22`	`{`
`26`		`- for (auto& buffer : m_pingPongBuffers)`
	`23`	`+ for (auto& buffer : m_pingPongBuffers.GetFramebuffers())`
`27`	`24`	`{`
`28`	`25`	`OvCore::Rendering::FramebufferUtil::SetupFramebuffer(`
`29`	`26`	`buffer, 1, 1, false, false, false`
`@@ -65,12 +62,8 @@ void OvCore::Rendering::PostProcessRenderPass::Draw(OvRendering::Data::PipelineS`
`65`	`62`
`66`	`63`	`if (auto stack = FindPostProcessStack(scene))`
`67`	`64`	`{`
`68`		`- uint32_t passIndex = 0;`
`69`		`-`
`70`	`65`	`auto& framebuffer = m_renderer.GetFrameDescriptor().outputBuffer.value();`
`71`	`66`
`72`		`- const uint64_t kPingPongBufferSize = m_pingPongBuffers.size();`
`73`		`-`
`74`	`67`	`m_renderer.Blit(p_pso, framebuffer, m_pingPongBuffers[0], m_blitMaterial);`
`75`	`68`
`76`	`69`	`for (auto& effect : m_effects)`
`@@ -81,17 +74,15 @@ void OvCore::Rendering::PostProcessRenderPass::Draw(OvRendering::Data::PipelineS`
`81`	`74`	`{`
`82`	`75`	`effect->Draw(`
`83`	`76`	`p_pso,`
`84`		`- m_pingPongBuffers[passIndex % kPingPongBufferSize],`
`85`		`- m_pingPongBuffers[(passIndex + 1) % kPingPongBufferSize],`
	`77`	`+ m_pingPongBuffers[0],`
	`78`	`+ m_pingPongBuffers[1],`
`86`	`79`	`settings`
`87`	`80`	`);`
`88`	`81`
`89`		`- ++passIndex;`
	`82`	`+ ++m_pingPongBuffers;`
`90`	`83`	`}`
`91`	`84`	`}`
`92`	`85`
`93`		`- const uint64_t lastIndex = passIndex % kPingPongBufferSize;`
`94`		`-`
`95`		`- m_renderer.Blit(p_pso, m_pingPongBuffers[lastIndex], framebuffer, m_blitMaterial);`
	`86`	`+ m_renderer.Blit(p_pso, m_pingPongBuffers[0], framebuffer, m_blitMaterial);`
`96`	`87`	`}`
`97`	`88`	`}`
Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,10 @@ OvRendering::Context::Driver::~Driver()`
`51`	`51`	`void OvRendering::Context::Driver::OnFrameCompleted()`
`52`	`52`	`{`
`53`	`53`	`m_gfxBackend->OnFrameCompleted();`
	`54`	`+`
	`55`	`+ // Prevents state leak between frames, and especially useful when external code (like ImGui)`
	`56`	`+ // requires a "neutral" pipeline state.`
	`57`	`+ ResetPipelineState();`
`54`	`58`	`}`
`55`	`59`
`56`	`60`	`void OvRendering::Context::Driver::SetViewport(uint32_t p_x, uint32_t p_y, uint32_t p_width, uint32_t p_height)`