Fix frame uniform binding in Metal rendering (AcademySoftwareFoundation#2415)

This PR adjusts how the frame uniform is bound in the Metal renderers of the Graph Editor and Viewer so that the frame index is correctly propagated to shaders.

Author: warrenm

source/MaterialXGraphEditor/RenderView.cpp

@@ -808,10 +808,7 @@ void RenderView::renderFrame()
         {
             material->getProgram()->bindUniform(mx::HW::ALPHA_THRESHOLD, mx::Value::createValue(0.99f));
         }
-        if (material->getProgram()->hasUniform(mx::HW::FRAME))
-        {
-            material->getProgram()->bindUniform(mx::HW::FRAME, mx::Value::createValue(static_cast<float>(_frame)));
-        }
+        material->getProgram()->bindTimeAndFrame(1.0f, static_cast<float>(_frame));
         material->bindViewInformation(_viewCamera);
         material->bindLighting(_lightHandler, _imageHandler, shadowState);
         material->bindImages(_imageHandler, _searchPath);
@@ -839,6 +836,7 @@ void RenderView::renderFrame()
             {
                 material->getProgram()->bindUniform(mx::HW::ALPHA_THRESHOLD, mx::Value::createValue(0.001f));
             }
+            material->getProgram()->bindTimeAndFrame(1.0f, static_cast<float>(_frame));
             material->bindViewInformation(_viewCamera);
             material->bindLighting(_lightHandler, _imageHandler, shadowState);
             material->bindImages(_imageHandler, _searchPath);

source/MaterialXRenderMsl/MslPipelineStateObject.mm

@@ -344,7 +344,6 @@ int GetStrideOfMetalType(MTLDataType type)
 
     // Bind based on inputs found
     bindViewInformation(cam);
-    bindTimeAndFrame();
     bindLighting(lightHandler, imageHandler);
     bindTextures(renderCmdEncoder, lightHandler, imageHandler);
     bindUniformBuffers(renderCmdEncoder, lightHandler, cam);

source/MaterialXView/RenderPipelineMetal.mm

@@ -542,10 +542,7 @@
         {
             material->getProgram()->bindUniform(mx::HW::ALPHA_THRESHOLD, mx::Value::createValue(0.99f));
         }
-        if (material->getProgram()->hasUniform(mx::HW::FRAME))
-        {
-            material->getProgram()->bindUniform(mx::HW::FRAME, mx::Value::createValue((float)_frame));
-        }
+        material->getProgram()->bindTimeAndFrame(1.0f, static_cast<float>(_frame));
         material->bindViewInformation(viewCamera);
         material->bindLighting(lightHandler, imageHandler, shadowState);
         material->bindImages(imageHandler, _viewer->_searchPath);
@@ -577,6 +574,7 @@
             {
                 material->getProgram()->bindUniform(mx::HW::ALPHA_THRESHOLD, mx::Value::createValue(0.001f));
             }
+            material->getProgram()->bindTimeAndFrame(1.0f, static_cast<float>(_frame));
             material->bindViewInformation(viewCamera);
             material->bindLighting(lightHandler, imageHandler, shadowState);
             material->bindImages(imageHandler, searchPath);