The motive of this PR is to break the big DrawPass function of the renderer, for this i decided to bring part of the function for instance

if (desc.drawWater)
{
	const auto& ocean = Locator::oceanSystem::value();
	const auto& mesh = ocean.GetMesh();
	mesh.GetIndexBuffer().Bind(mesh.GetIndexBuffer().GetCount(), 0);
	mesh.GetVertexBuffer().Bind();
	bgfx::setState(k_BgfxDefaultStateInvertedZ);
	auto diffuse = Locator::resources::value().GetTextures().Handle(ocean.GetDiffuseTexture());
	auto alpha = Locator::resources::value().GetTextures().Handle(ocean.GetAlphaTexture());
	waterShader->SetTextureSampler("s_diffuse", 0, *diffuse);
	waterShader->SetTextureSampler("s_alpha", 1, *alpha);
	waterShader->SetTextureSampler("s_reflection", 2, ocean.GetReflectionFramebuffer().GetColorAttachment());
	const glm::vec4 u_sky = {skyType, 0.0f, 0.0f, 0.0f};
	waterShader->SetUniformValue("u_sky", &u_sky); // fs
	bgfx::submit(static_cast<bgfx::ViewId>(desc.viewId), waterShader->GetRawHandle());
	k_TechniqueMap.at(Technique::Water)(desc);
}

into its own function.
To organise all this i would have all subtask of a pass be declared in a enumeration (i call Technique, you have one to draw the sky, one to draw entities, etc...) that would be mapped to variation of a templated function (named renderTechnique) in a way not to dissimilar with how we handle shader you would have to declare the Technique in the manage, and add the mapping to the function. Right now you also need to declare the instanciation but i hope to find a better way.

The long term objective would be :

• Have all part of Draw pass put into its own technique
• • At minimum getting rid of Mesh and transform stored inside the renderer (so sprites and debugCross can be separated)
• • Getting rid of DrawMesh and of dependency of Techniques from the renderer altogether
• Have drawdesc declare a set of techniques instead of having many boolean to tell if we draw entities or not
• • Have the renderer iterate through the set of techniques of the desc
• Adapt this to different pass as this doesn't fit well with how Footprint Pass are handled