Separate out the illumation of the scene

5 files changed, 102 insertions, 14 deletions

diff --git a/ui/gameMainWindow.cpp b/ui/gameMainWindow.cpp
index c31265a..0e861b4 100644
--- a/ui/gameMainWindow.cpp
+++ b/ui/gameMainWindow.cpp
@@ -45,9 +45,18 @@ GameMainWindow::tick(TickDuration)
 void
 GameMainWindow::render() const
 {
-	SceneRenderer::render([this](const auto & shader) {
-		gameState->world.apply<Renderable>(&Renderable::render, shader);
-		uiComponents.apply<WorldOverlay>(&WorldOverlay::render, shader);
-	});
+	SceneRenderer::render(*this);
 	Window::render();
 }
+void
+GameMainWindow::content(const SceneShader & shader) const
+{
+	gameState->world.apply<Renderable>(&Renderable::render, shader);
+	uiComponents.apply<WorldOverlay>(&WorldOverlay::render, shader);
+}
+void
+GameMainWindow::environment(const SceneShader & s, const SceneRenderer & r) const
+{
+	// default for now
+	SceneRenderer::SceneProvider::environment(s, r);
+}
diff --git a/ui/gameMainWindow.h b/ui/gameMainWindow.h
index 1e6af20..8485266 100644
--- a/ui/gameMainWindow.h
+++ b/ui/gameMainWindow.h
@@ -5,11 +5,15 @@
 #include "window.h"
 #include <cstddef>
 
-class GameMainWindow : public Window, SceneRenderer {
+class GameMainWindow : public Window, SceneRenderer, public SceneRenderer::SceneProvider {
 public:
 	GameMainWindow(size_t w, size_t h);
 
 	void tick(TickDuration) override;
 
 	void render() const override;
+
+private:
+	void content(const SceneShader &) const override;
+	void environment(const SceneShader &, const SceneRenderer &) const override;
 };
diff --git a/ui/sceneRenderer.cpp b/ui/sceneRenderer.cpp
index 6a68ac6..c3cff7a 100644
--- a/ui/sceneRenderer.cpp
+++ b/ui/sceneRenderer.cpp
@@ -1,5 +1,6 @@
 #include "sceneRenderer.h"
 #include "maths.h"
+#include <gfx/gl/shaders/fs-directionalLight.h>
 #include <gfx/gl/shaders/fs-lightingShader.h>
 #include <gfx/gl/shaders/vs-lightingShader.h>
 #include <glm/gtc/type_ptr.hpp>
@@ -34,9 +35,7 @@ SceneRenderer::SceneRenderer(glm::ivec2 s, GLuint o) :
 	configuregdata(gPosition, GL_RGBA16F, GL_FLOAT, GL_COLOR_ATTACHMENT0);
 	configuregdata(gNormal, GL_RGBA16F, GL_FLOAT, GL_COLOR_ATTACHMENT1);
 	configuregdata(gAlbedoSpec, GL_RGBA, GL_UNSIGNED_BYTE, GL_COLOR_ATTACHMENT2);
-	static constexpr std::array<unsigned int, 3> attachments {
-			GL_COLOR_ATTACHMENT0, GL_COLOR_ATTACHMENT1, GL_COLOR_ATTACHMENT2};
-	glDrawBuffers(attachments.size(), attachments.data());
+	configuregdata(gIllumination, GL_RGBA16F, GL_FLOAT, GL_COLOR_ATTACHMENT3);
 
 	glBindRenderbuffer(GL_RENDERBUFFER, depth);
 	glRenderbufferStorage(GL_RENDERBUFFER, GL_DEPTH_COMPONENT, size.x, size.y);
@@ -50,20 +49,35 @@ SceneRenderer::SceneRenderer(glm::ivec2 s, GLuint o) :
 }
 
 void
-SceneRenderer::render(std::function<void(const SceneShader &)> content) const
+SceneRenderer::render(const SceneProvider & scene) const
 {
 	shader.setView(camera.GetViewProjection());
 	glViewport(0, 0, size.x, size.y);
 
 	// Geometry pass
+	glBindFramebuffer(GL_FRAMEBUFFER, gBuffer);
+	static constexpr std::array<unsigned int, 3> attachments {
+			GL_COLOR_ATTACHMENT0, GL_COLOR_ATTACHMENT1, GL_COLOR_ATTACHMENT2};
+	glDrawBuffers(attachments.size(), attachments.data());
 	glEnable(GL_BLEND);
 	glEnable(GL_CULL_FACE);
 	glCullFace(GL_BACK);
 	glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);
 	glEnable(GL_DEPTH_TEST);
-	glBindFramebuffer(GL_FRAMEBUFFER, gBuffer);
+	glClearColor(0, 0, 0, 1);
 	glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT);
-	content(shader);
+	scene.content(shader);
+
+	// Illumination pass
+	glDrawBuffer(GL_COLOR_ATTACHMENT3);
+	glDisable(GL_DEPTH_TEST);
+	glEnable(GL_BLEND);
+	glBlendFunc(GL_ONE, GL_ONE);
+	glActiveTexture(GL_TEXTURE0);
+	glBindTexture(GL_TEXTURE_2D, gPosition);
+	glActiveTexture(GL_TEXTURE1);
+	glBindTexture(GL_TEXTURE_2D, gNormal);
+	scene.environment(shader, *this);
 
 	// Lighting pass
 	glBindFramebuffer(GL_FRAMEBUFFER, output);
@@ -75,9 +89,48 @@ SceneRenderer::render(std::function<void(const SceneShader &)> content) const
 	glBindTexture(GL_TEXTURE_2D, gNormal);
 	glActiveTexture(GL_TEXTURE2);
 	glBindTexture(GL_TEXTURE_2D, gAlbedoSpec);
-	// TODO Configure lights
+	glActiveTexture(GL_TEXTURE3);
+	glBindTexture(GL_TEXTURE_2D, gIllumination);
 	lighting.use();
 	glBindVertexArray(displayVAO);
 	glDrawArrays(GL_TRIANGLE_STRIP, 0, 4);
 	glBindVertexArray(0);
 }
+
+void
+SceneRenderer::setAmbientLight(const glm::vec3 & colour) const
+{
+	glClearColor(colour.r, colour.g, colour.b, 1.0F);
+	glClear(GL_COLOR_BUFFER_BIT);
+}
+
+void
+SceneRenderer::setDirectionalLight(const glm::vec3 & colour, const glm::vec3 & direction) const
+{
+	dirLight.use();
+	dirLight.setDirectionalLight(colour, direction);
+	glBindVertexArray(displayVAO);
+	glDrawArrays(GL_TRIANGLE_STRIP, 0, 4);
+	glBindVertexArray(0);
+}
+
+SceneRenderer::DirectionalLightProgram::DirectionalLightProgram() :
+	Program {lightingShader_vs, directionalLight_fs}, directionLoc {*this, "lightDirection"}, colourLoc {*this,
+																									  "lightColour"}
+{
+}
+
+void
+SceneRenderer::DirectionalLightProgram::setDirectionalLight(const glm::vec3 & c, const glm::vec3 & d) const
+{
+	glUniform3fv(colourLoc, 1, glm::value_ptr(c));
+	const auto nd = glm::normalize(d);
+	glUniform3fv(directionLoc, 1, glm::value_ptr(nd));
+}
+
+void
+SceneRenderer::SceneProvider::environment(const SceneShader &, const SceneRenderer & renderer) const
+{
+	renderer.setAmbientLight({0.5F, 0.5F, 0.5F});
+	renderer.setDirectionalLight({0.6F, 0.6F, 0.6F}, {1, 0, -1});
+}
diff --git a/ui/sceneRenderer.h b/ui/sceneRenderer.h
index c06fa83..8b81d0c 100644
--- a/ui/sceneRenderer.h
+++ b/ui/sceneRenderer.h
@@ -9,9 +9,18 @@
 
 class SceneRenderer {
 public:
+	class SceneProvider {
+	public:
+		virtual ~SceneProvider() = default;
+		virtual void content(const SceneShader &) const = 0;
+		virtual void environment(const SceneShader &, const SceneRenderer &) const;
+	};
+
 	explicit SceneRenderer(glm::ivec2 size, GLuint output);
 
-	void render(std::function<void(const SceneShader &)> content) const;
+	void render(const SceneProvider &) const;
+	void setAmbientLight(const glm::vec3 & colour) const;
+	void setDirectionalLight(const glm::vec3 & colour, const glm::vec3 & direction) const;
 
 	Camera camera;
 
@@ -19,14 +28,26 @@ private:
 	glm::ivec2 size;
 	GLuint output;
 	glFrameBuffer gBuffer;
-	glTexture gPosition, gNormal, gAlbedoSpec;
+	glTexture gPosition, gNormal, gAlbedoSpec, gIllumination;
 	glRenderBuffer depth;
 	class DeferredLightProgram : public Program {
 	public:
 		using Program::Program;
 		using Program::use;
 	};
+	class DirectionalLightProgram : public Program {
+	public:
+		DirectionalLightProgram();
+		using Program::use;
+
+		void setDirectionalLight(const glm::vec3 &, const glm::vec3 &) const;
+
+	private:
+		RequiredUniformLocation directionLoc, colourLoc;
+	};
+
 	DeferredLightProgram lighting;
+	DirectionalLightProgram dirLight;
 	glVertexArray displayVAO;
 	glBuffer displayVBO;
 	SceneShader shader;
diff --git a/ui/window.cpp b/ui/window.cpp
index 51aa827..5930292 100644
--- a/ui/window.cpp
+++ b/ui/window.cpp
@@ -77,6 +77,7 @@ void
 Window::render() const

 {

 	glEnable(GL_BLEND);

+	glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);

 	glDisable(GL_DEPTH_TEST);

 	uiComponents.apply(&UIComponent::render, uiShader, UIComponent::Position {});

 }