Forráskód Böngészése

Fix GLTFExport MorphTarget Bug

Chose a better candidate var for determining the true target node when merging morphTarget animation tracks.
Ammended comment to make more sense.
Bug-Reaper 5 éve
szülő
commit
84f5821946

+ 3 - 3
examples/js/exporters/GLTFExporter.js

@@ -2254,9 +2254,9 @@ THREE.GLTFExporter.Utils = {
 
 				}
 
-				// We need to take into consideration that the target node for a
-				// blendshape animation might not be the root node of the animation:
-				mergedTrack.name = sourceTrack.name;
+				// We need to take into consideration the intended target node
+				// of our original un-merged morphTarget animation.
+				mergedTrack.name = sourceTrackBinding.nodeName + '.morphTargetInfluences';
 				mergedTrack.values = values;
 
 				mergedTracks[ sourceTrackNode.uuid ] = mergedTrack;

+ 3 - 3
examples/jsm/exporters/GLTFExporter.js

@@ -2276,9 +2276,9 @@ GLTFExporter.Utils = {
 
 				}
 
-				// We need to take into consideration that the target node for a
-				// blendshape animation might not be the root node of the animation:
-				mergedTrack.name = sourceTrack.name
+				// We need to take into consideration the intended target node
+				// of our original un-merged morphTarget animation.
+				mergedTrack.name = sourceTrackBinding.nodeName + '.morphTargetInfluences';
 				mergedTrack.values = values;
 
 				mergedTracks[ sourceTrackNode.uuid ] = mergedTrack;