add option for not baking textures to file

to improve performance when the textures aren't being viewed
sideeffects · Aug 13, 2019 · 80a9016 · 80a9016
1 parent e802eea
commit 80a9016
Show file tree

Hide file tree

Showing 6 changed files with 23 additions and 6 deletions.
diff --git a/Asset.C b/Asset.C
@@ -936,6 +936,7 @@ void
 Asset::computeMaterial(
         const MPlug& plug,
         MDataBlock& data,
+	AssetNodeOptions::AccessorDataBlock &options,
         bool &needToSyncOutputs
         )
 {
@@ -957,7 +958,7 @@ Asset::computeMaterial(
         MPlug materialPlug = materialsPlug.elementByLogicalIndex(i);
         MDataHandle materialHandle = data.outputValue(materialPlug);
 
-        myMaterials[i]->compute(myTime, materialPlug, data, materialHandle);
+        myMaterials[i]->compute(myTime, materialPlug, data, materialHandle, options.bakeOutputTextures());
     }
 }
 
@@ -1142,7 +1143,9 @@ Asset::compute(
             options,
             needToSyncOutputs);
 
-    computeMaterial(plug, data, needToSyncOutputs);
+    computeMaterial(plug, data,
+            options,
+	    needToSyncOutputs);
 
     return stat;
 }

diff --git a/Asset.h b/Asset.h
@@ -94,6 +94,7 @@ class Asset {
         void computeMaterial(
                 const MPlug& plug,
                 MDataBlock& data,
+                AssetNodeOptions::AccessorDataBlock &options,
                 bool &needToSyncOutputs
                 );
 

diff --git a/AssetNodeOptions.h b/AssetNodeOptions.h
@@ -19,6 +19,7 @@ NODE_OPTION(outputMeshPreserveLockedNormals, bool, true)
 NODE_OPTION(ungroupOnBake, bool, true)
 NODE_OPTION(updateParmsForEvalMode, bool, true)
 NODE_OPTION(connectGeoForAssetInputs, bool, false)
+NODE_OPTION(bakeOutputTextures, bool, true)
 
 NODE_OPTION(useInstancerNode, bool, true)
 

diff --git a/OutputMaterial.C b/OutputMaterial.C
@@ -18,7 +18,8 @@ OutputMaterial::compute(
         const MTime &time,
         const MPlug &materialPlug,
         MDataBlock &data,
-        MDataHandle &materialHandle
+        MDataHandle &materialHandle,
+	bool bakeTexture
         )
 {
     HAPI_Result hapiResult;
@@ -155,9 +156,9 @@ OutputMaterial::compute(
 
             bool hasTextureSource = ((std::string)Util::HAPIString(texturePathSH)).size() > 0;
             bool canRenderTexture = false;
-            if(hasTextureSource)
+            if(hasTextureSource && bakeTexture)
             {
-                // this could fail if texture parameter is empty
+               // this could fail if texture parameter is empty
                 hapiResult = HAPI_RenderTextureToImage(
                         Util::theHAPISession.get(),
                         myNodeId,

diff --git a/OutputMaterial.h b/OutputMaterial.h
@@ -19,7 +19,8 @@ class OutputMaterial
             const MTime &time,
             const MPlug &materialPlug,
             MDataBlock &data,
-            MDataHandle &materialHandle
+            MDataHandle &materialHandle,
+	    bool bakeTexture
             );
 private:
     void update(MDataHandle &materialHandle);

diff --git a/scripts/AEhoudiniAssetTemplate.mel b/scripts/AEhoudiniAssetTemplate.mel
@@ -804,6 +804,11 @@ global proc AEhoudiniAssetOptionsNew(string $attr)
                 "Connect Geo For Asset Inputs",
                 "When setting an input to a geo output of another asset, connect through the geo, not directly from the asset"
                 );
+        createAttrCheckBox(
+                "bakeOutputTextures",
+                "Bake Output Textures",
+                "When an Output Material has a Texture, bake to a file texture. Turn this off to improve performace when the texture is not needed."
+                );
 
 
     setParent ..;
@@ -891,6 +896,10 @@ global proc AEhoudiniAssetOptionsReplace(string $attr)
             $optionsLayoutFull + "|outputLayout|connectGeoForAssetInputs",
             $nodeName + ".connectGeoForAssetInputs",
             "");
+    replaceAttrCheckBox(
+            $optionsLayoutFull + "|outputLayout|bakeOutputTextures",
+            $nodeName + ".bakeOutputTextures",
+            "");
     replaceAttrCheckBox($optionsLayoutFull + "|outputLayout|useInstancerNode",
             $nodeName + ".useInstancerNode",
             "houdiniEngine_syncAssetOutput \"" + $nodeName + "\";");
@@ -982,6 +991,7 @@ global proc AEhoudiniAssetTemplate(string $nodeName)
 	editorTemplate -suppress "output";
         editorTemplate -suppress "ungroupOnBake";
         editorTemplate -suppress "updateParmsForEvalMode";
+        editorTemplate -suppress "bakeOutputTextures";
         editorTemplate -suppress "connectGeoForAssetInputs";
 
         editorTemplate -suppress "useInstancerNode";