Skip to content

Commit

Permalink
Fix GLTFExport Bug with MorphTarget Animations which aren't aimed at …
Browse files Browse the repository at this point in the history
…the root node being exported:

Github Issue: #17579
  • Loading branch information
Bug-Reaper committed Apr 5, 2020
1 parent dcd1305 commit 2d3903e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
4 changes: 3 additions & 1 deletion examples/js/exporters/GLTFExporter.js
Expand Up @@ -2254,7 +2254,9 @@ THREE.GLTFExporter.Utils = {

}

mergedTrack.name = '.morphTargetInfluences';
// 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;
mergedTrack.values = values;

mergedTracks[ sourceTrackNode.uuid ] = mergedTrack;
Expand Down
4 changes: 3 additions & 1 deletion examples/jsm/exporters/GLTFExporter.js
Expand Up @@ -2276,7 +2276,9 @@ GLTFExporter.Utils = {

}

mergedTrack.name = '.morphTargetInfluences';
// 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
mergedTrack.values = values;

mergedTracks[ sourceTrackNode.uuid ] = mergedTrack;
Expand Down

0 comments on commit 2d3903e

Please sign in to comment.