static pipelines + misc render cleanup

libraries/render-utils/src/AmbientOcclusionEffect.cpp

@@ -842,6 +842,8 @@ void AmbientOcclusionEffect::run(const render::RenderContextPointer& renderConte
     config->setGPUBatchRunTime(_gpuTimer->getGPUAverage(), _gpuTimer->getBatchAverage());
 }
 
+gpu::PipelinePointer DebugAmbientOcclusion::_debugPipeline;
+
 void DebugAmbientOcclusion::configure(const Config& config) {
     _showCursorPixel = config.showCursorPixel;
 
@@ -851,7 +853,7 @@ void DebugAmbientOcclusion::configure(const Config& config) {
     }
 }
 
-const gpu::PipelinePointer& DebugAmbientOcclusion::getDebugPipeline() {
+gpu::PipelinePointer& DebugAmbientOcclusion::getDebugPipeline() {
     if (!_debugPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::ssao_debugOcclusion);
         gpu::StatePointer state = std::make_shared<gpu::State>();

libraries/render-utils/src/AmbientOcclusionEffect.h

@@ -283,9 +283,9 @@ private:
     };
     gpu::StructBuffer<Parameters> _parametersBuffer;
 
-    const gpu::PipelinePointer& getDebugPipeline();
+    static gpu::PipelinePointer& getDebugPipeline();
 
-    gpu::PipelinePointer _debugPipeline;
+    static gpu::PipelinePointer _debugPipeline;
 
     bool _showCursorPixel { false };
 };

libraries/render-utils/src/AntialiasingEffect.cpp

@@ -40,6 +40,9 @@ namespace gr {
 
 #if !ANTIALIASING_USE_TAA
 
+gpu::PipelinePointer Antialiasing::_antialiasingPipeline;
+gpu::PipelinePointer Antialiasing::_blendPipeline;
+
 Antialiasing::Antialiasing() {
     _geometryId = DependencyManager::get<GeometryCache>()->allocateID();
 }
@@ -144,6 +147,10 @@ void AntialiasingConfig::setAAMode(int mode) {
     emit dirty();
 }
 
+gpu::PipelinePointer Antialiasing::_antialiasingPipeline;
+gpu::PipelinePointer Antialiasing::_blendPipeline;
+gpu::PipelinePointer Antialiasing::_debugBlendPipeline;
+
 Antialiasing::Antialiasing(bool isSharpenEnabled) : 
     _isSharpenEnabled{ isSharpenEnabled } {
 }
@@ -154,7 +161,7 @@ Antialiasing::~Antialiasing() {
     _antialiasingTextures[1].reset();
 }
 
-const gpu::PipelinePointer& Antialiasing::getAntialiasingPipeline(const render::RenderContextPointer& renderContext) {
+gpu::PipelinePointer& Antialiasing::getAntialiasingPipeline() {
    
     if (!_antialiasingPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::taa);
@@ -169,7 +176,7 @@ const gpu::PipelinePointer& Antialiasing::getAntialiasingPipeline(const render::
     return _antialiasingPipeline;
 }
 
-const gpu::PipelinePointer& Antialiasing::getBlendPipeline() {
+gpu::PipelinePointer& Antialiasing::getBlendPipeline() {
     if (!_blendPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::fxaa_blend);
         gpu::StatePointer state = std::make_shared<gpu::State>();
@@ -180,7 +187,7 @@ const gpu::PipelinePointer& Antialiasing::getBlendPipeline() {
     return _blendPipeline;
 }
 
-const gpu::PipelinePointer& Antialiasing::getDebugBlendPipeline() {
+gpu::PipelinePointer& Antialiasing::getDebugBlendPipeline() {
     if (!_debugBlendPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::taa_blend);
         gpu::StatePointer state = std::make_shared<gpu::State>();
@@ -260,7 +267,6 @@ void Antialiasing::run(const render::RenderContextPointer& renderContext, const
         batch.setViewportTransform(args->_viewport);
 
         // TAA step
-        getAntialiasingPipeline(renderContext);
         batch.setResourceFramebufferSwapChainTexture(ru::Texture::TaaHistory, _antialiasingBuffers, 0);
         batch.setResourceTexture(ru::Texture::TaaSource, sourceBuffer->getRenderBuffer(0));
         batch.setResourceTexture(ru::Texture::TaaVelocity, velocityBuffer->getVelocityTexture());
@@ -269,9 +275,9 @@ void Antialiasing::run(const render::RenderContextPointer& renderContext, const
 
         batch.setUniformBuffer(ru::Buffer::TaaParams, _params);
         batch.setUniformBuffer(ru::Buffer::DeferredFrameTransform, deferredFrameTransform->getFrameTransformBuffer());
-        
+
         batch.setFramebufferSwapChain(_antialiasingBuffers, 1);
-        batch.setPipeline(getAntialiasingPipeline(renderContext));
+        batch.setPipeline(getAntialiasingPipeline());
         batch.draw(gpu::TRIANGLE_STRIP, 4);
 
         // Blend step

libraries/render-utils/src/AntialiasingEffect.h

@@ -213,18 +213,18 @@ public:
     void configure(const Config& config);
     void run(const render::RenderContextPointer& renderContext, const Inputs& inputs);
 
-    const gpu::PipelinePointer& getAntialiasingPipeline(const render::RenderContextPointer& renderContext);
+    static gpu::PipelinePointer& getAntialiasingPipeline();
-    const gpu::PipelinePointer& getBlendPipeline();
+    static gpu::PipelinePointer& getBlendPipeline();
-    const gpu::PipelinePointer& getDebugBlendPipeline();
+    static gpu::PipelinePointer& getDebugBlendPipeline();
 
 private:
 
     gpu::FramebufferSwapChainPointer _antialiasingBuffers;
     gpu::TexturePointer _antialiasingTextures[2];
     gpu::BufferPointer _blendParamsBuffer;
-    gpu::PipelinePointer _antialiasingPipeline;
+    static gpu::PipelinePointer _antialiasingPipeline;
-    gpu::PipelinePointer _blendPipeline;
+    static gpu::PipelinePointer _blendPipeline;
-    gpu::PipelinePointer _debugBlendPipeline;
+    static gpu::PipelinePointer _debugBlendPipeline;
 
     TAAParamsBuffer _params;
     AntialiasingConfig::Mode _mode{ AntialiasingConfig::TAA };
@@ -251,8 +251,8 @@ public:
     void configure(const Config& config) {}
     void run(const render::RenderContextPointer& renderContext, const gpu::FramebufferPointer& sourceBuffer);
     
-    const gpu::PipelinePointer& getAntialiasingPipeline();
+    static gpu::PipelinePointer& getAntialiasingPipeline();
-    const gpu::PipelinePointer& getBlendPipeline();
+    static gpu::PipelinePointer& getBlendPipeline();
 
 private:
     gpu::FramebufferPointer _antialiasingBuffer;
@@ -260,8 +260,8 @@ private:
     gpu::TexturePointer _antialiasingTexture;
     gpu::BufferPointer _paramsBuffer;
     
-    gpu::PipelinePointer _antialiasingPipeline;
+    static gpu::PipelinePointer _antialiasingPipeline;
-    gpu::PipelinePointer _blendPipeline;
+    static gpu::PipelinePointer _blendPipeline;
     int _geometryId { 0 };
 };
 #endif

libraries/render-utils/src/BloomEffect.cpp

@@ -19,6 +19,11 @@
 
 #define BLOOM_BLUR_LEVEL_COUNT  3
 
+gpu::PipelinePointer BloomThreshold::_pipeline;
+gpu::PipelinePointer BloomApply::_pipeline;
+gpu::PipelinePointer BloomDraw::_pipeline;
+gpu::PipelinePointer DebugBloom::_pipeline;
+
 BloomThreshold::BloomThreshold(unsigned int downsamplingFactor) {
     assert(downsamplingFactor > 0);
     _parameters.edit()._sampleCount = downsamplingFactor;

libraries/render-utils/src/BloomEffect.h

@@ -43,8 +43,8 @@ private:
 
 #include "BloomThreshold.shared.slh"
 
+    static gpu::PipelinePointer _pipeline;
     gpu::FramebufferPointer _outputBuffer;
-    gpu::PipelinePointer _pipeline;
     gpu::StructBuffer<Parameters> _parameters;
 };
 
@@ -68,7 +68,7 @@ private:
 
 #include "BloomApply.shared.slh"
 
-    gpu::PipelinePointer _pipeline;
+    static gpu::PipelinePointer _pipeline;
     gpu::StructBuffer<Parameters> _parameters;
 };
 
@@ -83,7 +83,7 @@ public:
 
 private:
 
-    gpu::PipelinePointer _pipeline;
+    static gpu::PipelinePointer _pipeline;
 };
 
 class DebugBloomConfig : public render::Job::Config {
@@ -125,7 +125,7 @@ public:
     void run(const render::RenderContextPointer& renderContext, const Inputs& inputs);
 
 private:
-    gpu::PipelinePointer _pipeline;
+    static gpu::PipelinePointer _pipeline;
     gpu::BufferPointer _params;
     DebugBloomConfig::Mode _mode;
 };

libraries/render-utils/src/DebugDeferredBuffer.cpp

@@ -250,6 +250,9 @@ static std::string getFileContent(const std::string& fileName, const std::string
     return defaultContent;
 }
 
+DebugDeferredBuffer::StandardPipelines DebugDeferredBuffer::_pipelines;
+DebugDeferredBuffer::CustomPipelines DebugDeferredBuffer::_customPipelines;
+
 #include <QStandardPaths>  // TODO REMOVE: Temporary until UI
 DebugDeferredBuffer::DebugDeferredBuffer() {
     // TODO REMOVE: Temporary until UI
@@ -334,7 +337,7 @@ std::string DebugDeferredBuffer::getShaderSourceCode(Mode mode, const std::strin
     return std::string();
 }
 
-bool DebugDeferredBuffer::pipelineNeedsUpdate(Mode mode, const std::string& customFile) const {
+bool DebugDeferredBuffer::pipelineNeedsUpdate(Mode mode, const std::string& customFile) {
     if (mode != CustomMode) {
         return !_pipelines[mode];
     }
@@ -351,7 +354,7 @@ bool DebugDeferredBuffer::pipelineNeedsUpdate(Mode mode, const std::string& cust
     return true;
 }
 
-const gpu::PipelinePointer& DebugDeferredBuffer::getPipeline(Mode mode, const std::string& customFile) {
+gpu::PipelinePointer& DebugDeferredBuffer::getPipeline(Mode mode, const std::string& customFile) {
     if (pipelineNeedsUpdate(mode, customFile)) {
         static_assert(shader::render_utils::program::debug_deferred_buffer != 0, "Validate debug deferred program");
 

libraries/render-utils/src/DebugDeferredBuffer.h

@@ -107,18 +107,18 @@ private:
 
     struct CustomPipeline {
         gpu::PipelinePointer pipeline;
-        mutable QFileInfo info;
+        QFileInfo info;
     };
     using StandardPipelines = std::array<gpu::PipelinePointer, NumModes>;
     using CustomPipelines = std::unordered_map<std::string, CustomPipeline>;
 
-    bool pipelineNeedsUpdate(Mode mode, const std::string& customFile = std::string()) const;
+    static bool pipelineNeedsUpdate(Mode mode, const std::string& customFile = std::string());
-    const gpu::PipelinePointer& getPipeline(Mode mode, const std::string& customFile = std::string());
+    static gpu::PipelinePointer& getPipeline(Mode mode, const std::string& customFile = std::string());
-    std::string getShaderSourceCode(Mode mode, const std::string& customFile = std::string());
+    static std::string getShaderSourceCode(Mode mode, const std::string& customFile = std::string());
 
     ParametersBuffer _parameters;
-    StandardPipelines _pipelines;
+    static StandardPipelines _pipelines;
-    CustomPipelines _customPipelines;
+    static CustomPipelines _customPipelines;
     int _geometryId{ 0 };
 };
 

libraries/render-utils/src/HighlightEffect.cpp

@@ -311,6 +311,8 @@ const gpu::PipelinePointer& DrawHighlight::getPipeline(const render::HighlightSt
     return style.isFilled() ? _pipelineFilled : _pipeline;
 }
 
+gpu::PipelinePointer DebugHighlight::_depthPipeline;
+
 DebugHighlight::DebugHighlight() {
     _geometryDepthId = DependencyManager::get<GeometryCache>()->allocateID();
 }
@@ -396,7 +398,7 @@ void DebugHighlight::initializePipelines() {
     _depthPipeline = gpu::Pipeline::create(program, state);
 }
 
-const gpu::PipelinePointer& DebugHighlight::getDepthPipeline() {
+gpu::PipelinePointer& DebugHighlight::getDepthPipeline() {
     if (!_depthPipeline) {
         initializePipelines();
     }

libraries/render-utils/src/HighlightEffect.h

@@ -186,12 +186,12 @@ public:
 
 private:
 
-    gpu::PipelinePointer _depthPipeline;
+    static gpu::PipelinePointer _depthPipeline;
     int _geometryDepthId{ 0 };
     bool _isDisplayEnabled{ false };
 
-    const gpu::PipelinePointer& getDepthPipeline();
+    static gpu::PipelinePointer& getDepthPipeline();
-    void initializePipelines();
+    static void initializePipelines();
 };
 
 class DrawHighlightTask {

libraries/render-utils/src/LightClusters.cpp

@@ -575,10 +575,9 @@ void LightClusteringPass::run(const render::RenderContextPointer& renderContext,
     config->setNumClusteredLightReferences(clusteringStats.z);
 }
 
-DebugLightClusters::DebugLightClusters() {
+gpu::PipelinePointer DebugLightClusters::_drawClusterGrid;
-
+gpu::PipelinePointer DebugLightClusters::_drawClusterFromDepth;
-}
+gpu::PipelinePointer DebugLightClusters::_drawClusterContent;
-
 
 void DebugLightClusters::configure(const Config& config) {
     doDrawGrid = config.doDrawGrid;
@@ -587,7 +586,7 @@ void DebugLightClusters::configure(const Config& config) {
 
 }
 
-const gpu::PipelinePointer DebugLightClusters::getDrawClusterGridPipeline() {
+gpu::PipelinePointer DebugLightClusters::getDrawClusterGridPipeline() {
     if (!_drawClusterGrid) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::lightClusters_drawGrid);
         auto state = std::make_shared<gpu::State>();
@@ -603,7 +602,7 @@ const gpu::PipelinePointer DebugLightClusters::getDrawClusterGridPipeline() {
     return _drawClusterGrid;
 }
 
-const gpu::PipelinePointer DebugLightClusters::getDrawClusterFromDepthPipeline() {
+gpu::PipelinePointer DebugLightClusters::getDrawClusterFromDepthPipeline() {
     if (!_drawClusterFromDepth) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::lightClusters_drawClusterFromDepth);
         auto state = std::make_shared<gpu::State>();
@@ -617,7 +616,7 @@ const gpu::PipelinePointer DebugLightClusters::getDrawClusterFromDepthPipeline()
     return _drawClusterFromDepth;
 }
 
-const gpu::PipelinePointer DebugLightClusters::getDrawClusterContentPipeline() {
+gpu::PipelinePointer DebugLightClusters::getDrawClusterContentPipeline() {
     if (!_drawClusterContent) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::lightClusters_drawClusterContent);
         auto state = std::make_shared<gpu::State>();

libraries/render-utils/src/LightClusters.h

@@ -217,23 +217,25 @@ public:
     using Config = DebugLightClustersConfig;
     using JobModel = render::Job::ModelI<DebugLightClusters, Inputs, Config>;
 
-    DebugLightClusters();
+    DebugLightClusters() {}
 
     void configure(const Config& config);
 
     void run(const render::RenderContextPointer& renderContext, const Inputs& inputs);
 
 protected:
+    static gpu::PipelinePointer _drawClusterGrid;
+    static gpu::PipelinePointer _drawClusterFromDepth;
+    static gpu::PipelinePointer _drawClusterContent;
     gpu::BufferPointer _gridBuffer;
-    gpu::PipelinePointer _drawClusterGrid;
+
-    gpu::PipelinePointer _drawClusterFromDepth;
-    gpu::PipelinePointer _drawClusterContent;
-    const gpu::PipelinePointer getDrawClusterGridPipeline();
-    const gpu::PipelinePointer getDrawClusterFromDepthPipeline();
-    const gpu::PipelinePointer getDrawClusterContentPipeline();
     bool doDrawGrid { false };
     bool doDrawClusterFromDepth { false };
     bool doDrawContent { false };
+
+    static gpu::PipelinePointer getDrawClusterGridPipeline();
+    static gpu::PipelinePointer getDrawClusterFromDepthPipeline();
+    static gpu::PipelinePointer getDrawClusterContentPipeline();
 };
 
 #endif

libraries/render-utils/src/StencilMaskPass.cpp

@@ -17,6 +17,10 @@
 
 using namespace render;
 
+gpu::PipelinePointer PrepareStencil::_meshStencilPipeline;
+gpu::PipelinePointer PrepareStencil::_paintStencilPipeline;
+graphics::MeshPointer PrepareStencil::_mesh;
+
 void PrepareStencil::configure(const Config& config) {
     _maskMode = config.maskMode;
 }

libraries/render-utils/src/StencilMaskPass.h

@@ -59,14 +59,14 @@ public:
     static void testMaskDrawShapeNoAA(gpu::State& state);
 
 private:
-    gpu::PipelinePointer _meshStencilPipeline;
+    static gpu::PipelinePointer _meshStencilPipeline;
-    gpu::PipelinePointer getMeshStencilPipeline();
+    static gpu::PipelinePointer getMeshStencilPipeline();
 
-    gpu::PipelinePointer _paintStencilPipeline;
+    static gpu::PipelinePointer _paintStencilPipeline;
-    gpu::PipelinePointer getPaintStencilPipeline();
+    static gpu::PipelinePointer getPaintStencilPipeline();
 
-    graphics::MeshPointer _mesh;
+    static graphics::MeshPointer _mesh;
-    graphics::MeshPointer getMesh();
+    static graphics::MeshPointer getMesh();
 
     StencilMaskMode _maskMode { StencilMaskMode::NONE };
 };

libraries/render-utils/src/SubsurfaceScattering.cpp

@@ -411,8 +411,8 @@ gpu::TexturePointer SubsurfaceScatteringResource::generateScatteringSpecularBeck
     return beckmannMap;
 }
 
-DebugSubsurfaceScattering::DebugSubsurfaceScattering() {
+gpu::PipelinePointer DebugSubsurfaceScattering::_scatteringPipeline;
-}
+gpu::PipelinePointer DebugSubsurfaceScattering::_showLUTPipeline;
 
 void DebugSubsurfaceScattering::configure(const Config& config) {
 
@@ -427,8 +427,6 @@ void DebugSubsurfaceScattering::configure(const Config& config) {
     _debugParams->setSubData(0, _debugCursorTexcoord);
 }
 
-
-
 gpu::PipelinePointer DebugSubsurfaceScattering::getScatteringPipeline() {
     if (!_scatteringPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::subsurfaceScattering_drawScattering);
@@ -440,8 +438,6 @@ gpu::PipelinePointer DebugSubsurfaceScattering::getScatteringPipeline() {
     return _scatteringPipeline;
 }
 
-gpu::PipelinePointer _showLUTPipeline;
-
 gpu::PipelinePointer DebugSubsurfaceScattering::getShowLUTPipeline() {
     if (!_showLUTPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::gpu::program::drawUnitQuatTextureOpaque);
@@ -452,14 +448,12 @@ gpu::PipelinePointer DebugSubsurfaceScattering::getShowLUTPipeline() {
     return _showLUTPipeline;
 }
 
-
 void DebugSubsurfaceScattering::run(const render::RenderContextPointer& renderContext, const Inputs& inputs) {
     assert(renderContext->args);
     assert(renderContext->args->hasViewFrustum());
 
     RenderArgs* args = renderContext->args;
 
-
     auto& frameTransform = inputs.get0();
     auto& deferredFramebuffer = inputs.get1();
 
@@ -477,8 +471,6 @@ void DebugSubsurfaceScattering::run(const render::RenderContextPointer& renderCo
     auto scatteringTable = scatteringResource->getScatteringTable();
     auto scatteringSpecular = scatteringResource->getScatteringSpecular();
 
-
-
     auto lightStage = renderContext->_scene->getStage<LightStage>();
     assert(lightStage);
    // const auto light = DependencyManager::get<DeferredLightingEffect>()->getLightStage()->getLight(0);
@@ -491,7 +483,6 @@ void DebugSubsurfaceScattering::run(const render::RenderContextPointer& renderCo
     gpu::doInBatch("DebugSubsurfaceScattering::run", args->_context, [=](gpu::Batch& batch) {
         batch.enableStereo(false);
 
-
         auto viewportSize = std::min(args->_viewport.z, args->_viewport.w) >> 1;
         auto offsetViewport = viewportSize * 0.1;
 

libraries/render-utils/src/SubsurfaceScattering.h

@@ -167,18 +167,19 @@ public:
     using Config = DebugSubsurfaceScatteringConfig;
     using JobModel = render::Job::ModelI<DebugSubsurfaceScattering, Inputs, Config>;
 
-    DebugSubsurfaceScattering();
+    DebugSubsurfaceScattering() {}
 
     void configure(const Config& config);
     void run(const render::RenderContextPointer& renderContext, const Inputs& inputs);
 
 private:
 
-    gpu::PipelinePointer _scatteringPipeline;
+    static gpu::PipelinePointer _scatteringPipeline;
-    gpu::PipelinePointer getScatteringPipeline();
+    static gpu::PipelinePointer getScatteringPipeline();
+
+    static gpu::PipelinePointer _showLUTPipeline;
+    static gpu::PipelinePointer getShowLUTPipeline();
 
-    gpu::PipelinePointer _showLUTPipeline;
-    gpu::PipelinePointer getShowLUTPipeline();
     gpu::BufferPointer _debugParams;
     bool _showProfile{ false };
     bool _showLUT{ false };

libraries/render-utils/src/SurfaceGeometryPass.cpp

@@ -137,9 +137,8 @@ gpu::TexturePointer LinearDepthFramebuffer::getHalfNormalTexture() {
     return _halfNormalTexture;
 }
 
-
+gpu::PipelinePointer LinearDepthPass::_linearDepthPipeline;
-LinearDepthPass::LinearDepthPass() {
+gpu::PipelinePointer LinearDepthPass::_downsamplePipeline;
-}
 
 void LinearDepthPass::configure(const Config& config) {
 }
@@ -179,8 +178,8 @@ void LinearDepthPass::run(const render::RenderContextPointer& renderContext, con
     outputs.edit3() = halfLinearDepthTexture;
     outputs.edit4() = halfNormalTexture;
    
-    auto linearDepthPipeline = getLinearDepthPipeline(renderContext);
+    auto linearDepthPipeline = getLinearDepthPipeline();
-    auto downsamplePipeline = getDownsamplePipeline(renderContext);
+    auto downsamplePipeline = getDownsamplePipeline();
 
     auto depthViewport = args->_viewport;
     auto halfViewport = depthViewport >> 1;
@@ -222,8 +221,7 @@ void LinearDepthPass::run(const render::RenderContextPointer& renderContext, con
     config->setGPUBatchRunTime(_gpuTimer->getGPUAverage(), _gpuTimer->getBatchAverage());
 }
 
-
+const gpu::PipelinePointer& LinearDepthPass::getLinearDepthPipeline() {
-const gpu::PipelinePointer& LinearDepthPass::getLinearDepthPipeline(const render::RenderContextPointer& renderContext) {
     gpu::ShaderPointer program;
     if (!_linearDepthPipeline) {
         program = gpu::Shader::createProgram(shader::render_utils::program::surfaceGeometry_makeLinearDepth);
@@ -243,8 +241,7 @@ const gpu::PipelinePointer& LinearDepthPass::getLinearDepthPipeline(const render
     return _linearDepthPipeline;
 }
 
-
+const gpu::PipelinePointer& LinearDepthPass::getDownsamplePipeline() {
-const gpu::PipelinePointer& LinearDepthPass::getDownsamplePipeline(const render::RenderContextPointer& renderContext) {
     if (!_downsamplePipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::surfaceGeometry_downsampleDepthNormal);
 
@@ -366,6 +363,8 @@ void SurfaceGeometryFramebuffer::setResolutionLevel(int resolutionLevel) {
     }
 }
 
+gpu::PipelinePointer SurfaceGeometryPass::_curvaturePipeline;
+
 SurfaceGeometryPass::SurfaceGeometryPass() :
     _diffusePass(false)
 {
@@ -454,7 +453,7 @@ void SurfaceGeometryPass::run(const render::RenderContextPointer& renderContext,
     outputs.edit2() = curvatureFramebuffer;
     outputs.edit3() = lowCurvatureFramebuffer;
 
-    auto curvaturePipeline = getCurvaturePipeline(renderContext);
+    auto curvaturePipeline = getCurvaturePipeline();
     auto diffuseVPipeline = _diffusePass.getBlurVPipeline();
     auto diffuseHPipeline = _diffusePass.getBlurHPipeline();
 
@@ -532,7 +531,7 @@ void SurfaceGeometryPass::run(const render::RenderContextPointer& renderContext,
     config->setGPUBatchRunTime(_gpuTimer->getGPUAverage(), _gpuTimer->getBatchAverage());
 }
 
-const gpu::PipelinePointer& SurfaceGeometryPass::getCurvaturePipeline(const render::RenderContextPointer& renderContext) {
+const gpu::PipelinePointer& SurfaceGeometryPass::getCurvaturePipeline() {
     if (!_curvaturePipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::surfaceGeometry_makeCurvature);
 

libraries/render-utils/src/SurfaceGeometryPass.h

@@ -73,21 +73,19 @@ public:
     using Config = LinearDepthPassConfig;
     using JobModel = render::Job::ModelIO<LinearDepthPass, Inputs, Outputs, Config>;
 
-    LinearDepthPass();
+    LinearDepthPass() {}
 
     void configure(const Config& config);
     void run(const render::RenderContextPointer& renderContext, const Inputs& inputs, Outputs& outputs);
     
 private:
-    typedef gpu::BufferView UniformBufferView;
-
     LinearDepthFramebufferPointer _linearDepthFramebuffer;
 
-    const gpu::PipelinePointer& getLinearDepthPipeline(const render::RenderContextPointer& renderContext);
+    static const gpu::PipelinePointer& getLinearDepthPipeline();
-    gpu::PipelinePointer _linearDepthPipeline;
+    static gpu::PipelinePointer _linearDepthPipeline;
 
-    const gpu::PipelinePointer& getDownsamplePipeline(const render::RenderContextPointer& renderContext);
+    static const gpu::PipelinePointer& getDownsamplePipeline();
-    gpu::PipelinePointer _downsamplePipeline;
+    static gpu::PipelinePointer _downsamplePipeline;
 
     gpu::RangeTimerPointer _gpuTimer;
 };
@@ -194,16 +192,14 @@ private:
     };
     gpu::BufferView _parametersBuffer;
 
-
     SurfaceGeometryFramebufferPointer _surfaceGeometryFramebuffer;
 
-    const gpu::PipelinePointer& getCurvaturePipeline(const render::RenderContextPointer& renderContext);
+    static const gpu::PipelinePointer& getCurvaturePipeline();
+
+    static gpu::PipelinePointer _curvaturePipeline;
 
-    gpu::PipelinePointer _curvaturePipeline;
-    
     render::BlurGaussianDepthAware _diffusePass;
 
-
     gpu::RangeTimerPointer _gpuTimer;
 };
 

libraries/render-utils/src/ToneMapAndResampleTask.h

@@ -67,7 +67,6 @@ protected:
     gpu::FramebufferPointer _destinationFrameBuffer;
 
 private:
-    gpu::PipelinePointer _blitLightBuffer;
     float _exposure { 0.0f };
 
     bool _debug { false };

libraries/render-utils/src/VelocityBufferPass.cpp

@@ -81,8 +81,7 @@ gpu::TexturePointer VelocityFramebuffer::getVelocityTexture() {
     return _velocityTexture;
 }
 
-VelocityBufferPass::VelocityBufferPass() {
+gpu::PipelinePointer VelocityBufferPass::_cameraMotionPipeline;
-}
 
 void VelocityBufferPass::configure(const Config& config) {
 }
@@ -114,7 +113,7 @@ void VelocityBufferPass::run(const render::RenderContextPointer& renderContext,
     outputs.edit1() = velocityFBO;
     outputs.edit2() = velocityTexture;
    
-    auto cameraMotionPipeline = getCameraMotionPipeline(renderContext);
+    auto cameraMotionPipeline = getCameraMotionPipeline();
 
     auto fullViewport = args->_viewport;
 
@@ -144,7 +143,7 @@ void VelocityBufferPass::run(const render::RenderContextPointer& renderContext,
 }
 
 
-const gpu::PipelinePointer& VelocityBufferPass::getCameraMotionPipeline(const render::RenderContextPointer& renderContext) {
+const gpu::PipelinePointer& VelocityBufferPass::getCameraMotionPipeline() {
     if (!_cameraMotionPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::velocityBuffer_cameraMotion);
         gpu::StatePointer state = std::make_shared<gpu::State>();

libraries/render-utils/src/VelocityBufferPass.h

@@ -69,7 +69,7 @@ public:
     using Config = VelocityBufferPassConfig;
     using JobModel = render::Job::ModelIO<VelocityBufferPass, Inputs, Outputs, Config>;
 
-    VelocityBufferPass();
+    VelocityBufferPass() {}
 
     void configure(const Config& config);
     void run(const render::RenderContextPointer& renderContext, const Inputs& inputs, Outputs& outputs);
@@ -79,8 +79,8 @@ private:
 
     VelocityFramebufferPointer _velocityFramebuffer;
 
-    const gpu::PipelinePointer& getCameraMotionPipeline(const render::RenderContextPointer& renderContext);
+    static const gpu::PipelinePointer& getCameraMotionPipeline();
-    gpu::PipelinePointer _cameraMotionPipeline;
+    static gpu::PipelinePointer _cameraMotionPipeline;
 
     gpu::RangeTimerPointer _gpuTimer;
 };

libraries/render-utils/src/ZoneRenderer.cpp

@@ -93,6 +93,10 @@ void SetupZones::run(const RenderContextPointer& context, const Input& input) {
     ambientOcclusionStage->_currentFrame.pushAmbientOcclusion(INVALID_INDEX);
 }
 
+gpu::PipelinePointer DebugZoneLighting::_keyLightPipeline;
+gpu::PipelinePointer DebugZoneLighting::_ambientPipeline;
+gpu::PipelinePointer DebugZoneLighting::_backgroundPipeline;
+
 const gpu::PipelinePointer& DebugZoneLighting::getKeyLightPipeline() {
     if (!_keyLightPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render_utils::program::zone_drawKeyLight);

libraries/render-utils/src/ZoneRenderer.h

@@ -83,13 +83,13 @@ public:
 
 protected:
 
-    gpu::PipelinePointer _keyLightPipeline;
+    static gpu::PipelinePointer _keyLightPipeline;
-    gpu::PipelinePointer _ambientPipeline;
+    static gpu::PipelinePointer _ambientPipeline;
-    gpu::PipelinePointer _backgroundPipeline;
+    static gpu::PipelinePointer _backgroundPipeline;
 
-    const gpu::PipelinePointer& getKeyLightPipeline();
+    static const gpu::PipelinePointer& getKeyLightPipeline();
-    const gpu::PipelinePointer& getAmbientPipeline();
+    static const gpu::PipelinePointer& getAmbientPipeline();
-    const gpu::PipelinePointer& getBackgroundPipeline();
+    static const gpu::PipelinePointer& getBackgroundPipeline();
 };
 
 

libraries/render/src/render/BlurTask.cpp

@@ -195,6 +195,9 @@ bool BlurInOutResource::updateResources(const gpu::FramebufferPointer& sourceFra
     return true;
 }
 
+gpu::PipelinePointer BlurGaussian::_blurVPipeline;
+gpu::PipelinePointer BlurGaussian::_blurHPipeline;
+
 BlurGaussian::BlurGaussian() {
     _parameters = std::make_shared<BlurParams>();
 }
@@ -294,7 +297,8 @@ void BlurGaussian::run(const RenderContextPointer& renderContext, const Inputs&
     });
 }
 
-
+gpu::PipelinePointer BlurGaussianDepthAware::_blurVPipeline;
+gpu::PipelinePointer BlurGaussianDepthAware::_blurHPipeline;
 
 BlurGaussianDepthAware::BlurGaussianDepthAware(bool generateOutputFramebuffer, const BlurParamsPointer& params) :
     _inOutResources(generateOutputFramebuffer, 1U),

libraries/render/src/render/BlurTask.h

@@ -129,11 +129,11 @@ protected:
 
     BlurParamsPointer _parameters;
 
-    gpu::PipelinePointer _blurVPipeline;
+    static gpu::PipelinePointer _blurVPipeline;
-    gpu::PipelinePointer _blurHPipeline;
+    static gpu::PipelinePointer _blurHPipeline;
 
-    gpu::PipelinePointer getBlurVPipeline();
+    static gpu::PipelinePointer getBlurVPipeline();
-    gpu::PipelinePointer getBlurHPipeline();
+    static gpu::PipelinePointer getBlurHPipeline();
 
     BlurInOutResource _inOutResources;
 };
@@ -163,12 +163,12 @@ public:
 
     const BlurParamsPointer& getParameters() const { return _parameters; }
  
-    gpu::PipelinePointer getBlurVPipeline();
+    static gpu::PipelinePointer getBlurVPipeline();
-    gpu::PipelinePointer getBlurHPipeline();
+    static gpu::PipelinePointer getBlurHPipeline();
 
 protected:
-    gpu::PipelinePointer _blurVPipeline;
+    static gpu::PipelinePointer _blurVPipeline;
-    gpu::PipelinePointer _blurHPipeline;
+    static gpu::PipelinePointer _blurHPipeline;
 
     BlurInOutResource _inOutResources;
     BlurParamsPointer _parameters;

libraries/render/src/render/DrawSceneOctree.cpp

@@ -24,6 +24,15 @@
 
 using namespace render;
 
+gpu::PipelinePointer DrawSceneOctree::_drawCellBoundsPipeline;
+gpu::PipelinePointer DrawSceneOctree::_drawLODReticlePipeline;
+gpu::PipelinePointer DrawSceneOctree::_drawItemBoundPipeline;
+gpu::Stream::FormatPointer DrawSceneOctree::_cellBoundsFormat = std::make_shared<gpu::Stream::Format>();
+
+DrawSceneOctree::DrawSceneOctree() {
+    _cellBoundsFormat->setAttribute(0, 0, gpu::Element(gpu::VEC4, gpu::INT32, gpu::XYZW), 0, gpu::Stream::PER_INSTANCE);
+}
+
 const gpu::PipelinePointer DrawSceneOctree::getDrawCellBoundsPipeline() {
     if (!_drawCellBoundsPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render::program::drawCellBounds);
@@ -36,9 +45,6 @@ const gpu::PipelinePointer DrawSceneOctree::getDrawCellBoundsPipeline() {
 
         // Good to go add the brand new pipeline
         _drawCellBoundsPipeline = gpu::Pipeline::create(program, state);
-        _cellBoundsFormat = std::make_shared<gpu::Stream::Format>();
-        _cellBoundsFormat->setAttribute(0, 0, gpu::Element(gpu::VEC4, gpu::INT32, gpu::XYZW), 0, gpu::Stream::PER_INSTANCE);
-        _cellBoundsBuffer = std::make_shared<gpu::Buffer>();
     }
     return _drawCellBoundsPipeline;
 }
@@ -62,7 +68,6 @@ void DrawSceneOctree::configure(const Config& config) {
     _showEmptyCells = config.showEmptyCells;
 }
 
-
 void DrawSceneOctree::run(const RenderContextPointer& renderContext, const ItemSpatialTree::ItemSelection& inSelection) {
     assert(renderContext->args);
     assert(renderContext->args->hasViewFrustum());
@@ -132,6 +137,8 @@ void DrawSceneOctree::run(const RenderContextPointer& renderContext, const ItemS
     });
 }
 
+gpu::PipelinePointer DrawItemSelection::_drawItemBoundPipeline;
+
 const gpu::PipelinePointer DrawItemSelection::getDrawItemBoundPipeline() {
     if (!_drawItemBoundPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render::program::drawItemBounds);

libraries/render/src/render/DrawSceneOctree.h

@@ -50,11 +50,11 @@ namespace render {
     };
 
     class DrawSceneOctree {
-        gpu::PipelinePointer _drawCellBoundsPipeline;
+        static gpu::PipelinePointer _drawCellBoundsPipeline;
-        gpu::PipelinePointer _drawLODReticlePipeline;
+        static gpu::PipelinePointer _drawLODReticlePipeline;
-        gpu::PipelinePointer _drawItemBoundPipeline;
+        static gpu::PipelinePointer _drawItemBoundPipeline;
-        gpu::BufferPointer _cellBoundsBuffer;
+        static gpu::Stream::FormatPointer _cellBoundsFormat;
-        gpu::Stream::FormatPointer _cellBoundsFormat;
+        gpu::BufferPointer _cellBoundsBuffer { std::make_shared<gpu::Buffer>() };
 
         bool _showVisibleCells; // initialized by Config
         bool _showEmptyCells; // initialized by Config
@@ -63,14 +63,14 @@ namespace render {
         using Config = DrawSceneOctreeConfig;
         using JobModel = Job::ModelI<DrawSceneOctree, ItemSpatialTree::ItemSelection, Config>;
 
-        DrawSceneOctree() {}
+        DrawSceneOctree();
 
         void configure(const Config& config);
         void run(const RenderContextPointer& renderContext, const ItemSpatialTree::ItemSelection& selection);
 
-        const gpu::PipelinePointer getDrawCellBoundsPipeline();
+        static const gpu::PipelinePointer getDrawCellBoundsPipeline();
-        const gpu::PipelinePointer getDrawLODReticlePipeline();
+        static const gpu::PipelinePointer getDrawLODReticlePipeline();
-        const gpu::PipelinePointer getDrawItemBoundPipeline();
+        static const gpu::PipelinePointer getDrawItemBoundPipeline();
     };
 
 
@@ -105,7 +105,7 @@ namespace render {
     };
 
     class DrawItemSelection {
-        gpu::PipelinePointer _drawItemBoundPipeline;
+        static gpu::PipelinePointer _drawItemBoundPipeline;
         gpu::BufferPointer _boundsBufferInside;
         gpu::BufferPointer _boundsBufferInsideSubcell;
         gpu::BufferPointer _boundsBufferPartial;
@@ -125,7 +125,7 @@ namespace render {
         void configure(const Config& config);
         void run(const RenderContextPointer& renderContext, const ItemSpatialTree::ItemSelection& selection);
 
-        const gpu::PipelinePointer getDrawItemBoundPipeline();
+        static const gpu::PipelinePointer getDrawItemBoundPipeline();
     };
 }
 

libraries/render/src/render/DrawStatus.cpp

@@ -32,6 +32,10 @@ void DrawStatusConfig::dirtyHelper() {
     emit dirty();
 }
 
+gpu::PipelinePointer DrawStatus::_drawItemBoundsPipeline;
+gpu::PipelinePointer DrawStatus::_drawItemStatusPipeline;
+gpu::Stream::FormatPointer DrawStatus::_vertexFormat;
+
 const gpu::PipelinePointer DrawStatus::getDrawItemBoundsPipeline() {
     if (!_drawItemBoundsPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render::program::drawItemBounds);

libraries/render/src/render/DrawStatus.h

@@ -52,8 +52,8 @@ namespace render {
         void configure(const Config& config);
         void run(const RenderContextPointer& renderContext, const Input& input);
 
-        const gpu::PipelinePointer getDrawItemBoundsPipeline();
+        static const gpu::PipelinePointer getDrawItemBoundsPipeline();
-        const gpu::PipelinePointer getDrawItemStatusPipeline();
+        static const gpu::PipelinePointer getDrawItemStatusPipeline();
 
         void setStatusIconMap(const gpu::TexturePointer& map);
         const gpu::TexturePointer getStatusIconMap() const;
@@ -63,13 +63,12 @@ namespace render {
         bool _showNetwork { false }; // initialized by Config
         bool _showFade { false }; // initialized by Config
 
-        gpu::Stream::FormatPointer _drawItemFormat;
+        static gpu::PipelinePointer _drawItemBoundsPipeline;
-        gpu::PipelinePointer _drawItemBoundsPipeline;
+        static gpu::PipelinePointer _drawItemStatusPipeline;
-        gpu::PipelinePointer _drawItemStatusPipeline;
+        static gpu::Stream::FormatPointer _vertexFormat;
 
         gpu::BufferPointer _boundsBuffer;
         gpu::BufferPointer _instanceBuffer;
-        gpu::Stream::FormatPointer _vertexFormat;
         gpu::TexturePointer _statusIconMap;
     };
 }

libraries/render/src/render/DrawTask.cpp

@@ -155,6 +155,8 @@ void DrawLight::run(const RenderContextPointer& renderContext, const ItemBounds&
     config->setNumDrawn((int)inLights.size());
 }
 
+gpu::PipelinePointer DrawBounds::_boundsPipeline;
+
 const gpu::PipelinePointer DrawBounds::getPipeline() {
     if (!_boundsPipeline) {
         gpu::ShaderPointer program = gpu::Shader::createProgram(shader::render::program::drawItemBounds);

libraries/render/src/render/DrawTask.h

@@ -63,8 +63,8 @@ public:
         const Inputs& items);
 
 private:
-    const gpu::PipelinePointer getPipeline();
+    static const gpu::PipelinePointer getPipeline();
-    gpu::PipelinePointer _boundsPipeline;
+    static gpu::PipelinePointer _boundsPipeline;
     gpu::BufferPointer _drawBuffer;
     gpu::BufferPointer _paramsBuffer;
 };

libraries/task/src/task/Task.h

@@ -332,7 +332,7 @@ public:
         void run(const ContextPointer& jobContext) override {
             auto config = std::static_pointer_cast<C>(Concept::_config);
             if (config->isEnabled()) {
-                for (auto job : TaskConcept::_jobs) {
+                for (auto& job : TaskConcept::_jobs) {
                     job.run(jobContext);
                     if (jobContext->taskFlow.doAbortTask()) {
                         jobContext->taskFlow.reset();

scripts/developer/utilities/lib/jet/jet.js

@@ -100,7 +100,6 @@ function job_print_functor(printout, showProps, showInOuts, maxDepth) {
             }
         }
         if (showInOuts) {
-            printout("jsdkfkjdskflj")
             var inouts = job_inoutKeys(job);
             for (var p=0; p < inouts.length;p++) {
                 var prop = job[inouts[p]]