Fixing a crash related to the new requirement of MaterialX to support…

… tangetnt buffer.
Autodesk · Mar 15, 2022 · fdcfcff · fdcfcff
1 parent 9d4021c
commit fdcfcff
Showing 1 changed file with 15 additions and 0 deletions.
diff --git a/lib/mayaUsd/render/vp2RenderDelegate/points.cpp b/lib/mayaUsd/render/vp2RenderDelegate/points.cpp
@@ -46,6 +46,7 @@ TF_DEFINE_PRIVATE_TOKENS(
     _tokens,
 
     (spriteWidth)
+    (tangents)
 );
 // clang-format on
 
@@ -327,6 +328,20 @@ void HdVP2Points::_UpdateDrawItem(
                     _CommitMVertexBuffer(_pointsSharedData._normalsBuffer.get(), bufferData);
                 }
             }
+
+            // Some materials, particularly MaterialX, require tangents data to be present
+            // in the vertex buffers. So for now, to avoid crashes, let's add dummy tungents.
+            // We may need to reevaluate it later.
+            const MHWRender::MVertexBufferDescriptor vbDesc(
+                "", MHWRender::MGeometry::kTangent, MHWRender::MGeometry::kFloat, 3);
+            VtVec3fArray tangents;
+            PreparePrimvarBuffer(
+                _pointsSharedData,
+                stateToCommit,
+                _tokens->tangents,
+                _tokens->tangents,
+                vbDesc,
+                GfVec3f(1.f, 0.f, 0.f));
         }
 
         // Prepare primvar buffers.