@sage-rsc/talking-head-react 1.4.5 → 1.4.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sage-rsc/talking-head-react",
-  "version": "1.4.5",
+  "version": "1.4.7",
   "description": "A reusable React component for 3D talking avatars with lip-sync and text-to-speech",
   "main": "./dist/index.cjs",
   "module": "./dist/index.js",

package/src/lib/talkinghead.mjs

@@ -1619,8 +1619,8 @@ class TalkingHead {
    */
   applyShoulderAdjustment() {
     const tempEuler = new THREE.Euler();
-    const targetX = 0.8; // Target X rotation for relaxed shoulders
-    const maxX = 0.9; // Maximum X rotation
+    const targetX = 0.6; // Target X rotation for relaxed shoulders - lowered significantly
+    const maxX = 0.7; // Maximum X rotation - lowered significantly
     
     // Adjust left shoulder
     if (this.poseAvatar.props['LeftShoulder.quaternion']) {
@@ -1630,7 +1630,7 @@ class TalkingHead {
       if (tempEuler.x > maxX) {
         tempEuler.x = targetX;
       } else if (tempEuler.x > targetX) {
-        tempEuler.x = targetX + (tempEuler.x - targetX) * 0.3;
+        tempEuler.x = targetX + (tempEuler.x - targetX) * 0.2; // More aggressive reduction
       }
       leftShoulder.setFromEuler(tempEuler, 'XYZ');
     }
@@ -1643,7 +1643,7 @@ class TalkingHead {
       if (tempEuler.x > maxX) {
         tempEuler.x = targetX;
       } else if (tempEuler.x > targetX) {
-        tempEuler.x = targetX + (tempEuler.x - targetX) * 0.3;
+        tempEuler.x = targetX + (tempEuler.x - targetX) * 0.2; // More aggressive reduction
       }
       rightShoulder.setFromEuler(tempEuler, 'XYZ');
     }
@@ -1668,15 +1668,16 @@ class TalkingHead {
     const tempQuaternion = new THREE.Quaternion();
     
     // Research-based relaxed shoulder rotation values
-    // Natural relaxed shoulders: X rotation ~0.7-0.9 radians
+    // Natural relaxed shoulders: X rotation ~0.5-0.7 radians (much lower for natural look)
     // High/stiff shoulders: X rotation ~1.5-1.8 radians
-    // We clamp to a natural relaxed position
-    const targetX = 0.8; // Target X rotation for relaxed, natural shoulders (radians)
-    const maxX = 0.9; // Maximum allowed X rotation
+    // We aggressively clamp to a very relaxed position
+    const targetX = 0.6; // Target X rotation for relaxed, natural shoulders (radians) - lowered significantly
+    const maxX = 0.7; // Maximum allowed X rotation - lowered significantly
     
     // Adjust left shoulder bone directly
     if (leftShoulderBone.quaternion) {
       tempEuler.setFromQuaternion(leftShoulderBone.quaternion, 'XYZ');
+      const originalX = tempEuler.x;
       
       // Aggressively clamp X rotation to relaxed position
       if (tempEuler.x > maxX) {
@@ -1684,17 +1685,21 @@ class TalkingHead {
         tempEuler.x = targetX;
       } else if (tempEuler.x > targetX) {
         // Smoothly reduce if slightly above target
-        tempEuler.x = targetX + (tempEuler.x - targetX) * 0.3;
+        tempEuler.x = targetX + (tempEuler.x - targetX) * 0.2; // More aggressive reduction
       }
       
-      tempQuaternion.setFromEuler(tempEuler, 'XYZ');
-      leftShoulderBone.quaternion.copy(tempQuaternion);
-      leftShoulderBone.updateMatrixWorld(true);
+      // Only update if we actually changed something
+      if (Math.abs(tempEuler.x - originalX) > 0.01) {
+        tempQuaternion.setFromEuler(tempEuler, 'XYZ');
+        leftShoulderBone.quaternion.copy(tempQuaternion);
+        leftShoulderBone.updateMatrixWorld(true);
+      }
     }
     
     // Adjust right shoulder bone directly
     if (rightShoulderBone.quaternion) {
       tempEuler.setFromQuaternion(rightShoulderBone.quaternion, 'XYZ');
+      const originalX = tempEuler.x;
       
       // Aggressively clamp X rotation to relaxed position
       if (tempEuler.x > maxX) {
@@ -1702,12 +1707,15 @@ class TalkingHead {
         tempEuler.x = targetX;
       } else if (tempEuler.x > targetX) {
         // Smoothly reduce if slightly above target
-        tempEuler.x = targetX + (tempEuler.x - targetX) * 0.3;
+        tempEuler.x = targetX + (tempEuler.x - targetX) * 0.2; // More aggressive reduction
       }
       
-      tempQuaternion.setFromEuler(tempEuler, 'XYZ');
-      rightShoulderBone.quaternion.copy(tempQuaternion);
-      rightShoulderBone.updateMatrixWorld(true);
+      // Only update if we actually changed something
+      if (Math.abs(tempEuler.x - originalX) > 0.01) {
+        tempQuaternion.setFromEuler(tempEuler, 'XYZ');
+        rightShoulderBone.quaternion.copy(tempQuaternion);
+        rightShoulderBone.updateMatrixWorld(true);
+      }
     }
   }
 
@@ -5752,6 +5760,7 @@ class TalkingHead {
         // Filter and map animation tracks
         const mappedTracks = [];
         const unmappedBones = new Set();
+        let filteredShoulderTracks = 0;
         anim.tracks.forEach(track => {
           // Remove mixamorig prefix first
           let trackName = track.name.replaceAll('mixamorig', '');
@@ -5762,6 +5771,14 @@ class TalkingHead {
           // Map bone name to avatar skeleton
           const mappedBoneName = mapBoneName(fbxBoneName);
           
+          // Filter out shoulder rotation tracks - we'll control shoulders manually
+          if (mappedBoneName && (mappedBoneName === 'LeftShoulder' || mappedBoneName === 'RightShoulder')) {
+            if (property === 'quaternion' || property === 'rotation') {
+              filteredShoulderTracks++;
+              return; // Skip this track - don't add it to mappedTracks
+            }
+          }
+          
           if (mappedBoneName && property) {
             // Create new track with mapped bone name
             const newTrackName = `${mappedBoneName}.${property}`;
@@ -5789,6 +5806,10 @@ class TalkingHead {
           }
         });
         
+        if (filteredShoulderTracks > 0) {
+          console.log(`✓ Filtered out ${filteredShoulderTracks} shoulder rotation track(s) to prevent high shoulders`);
+        }
+        
         if (unmappedBones.size > 0) {
           console.warn(`⚠️ ${unmappedBones.size} bone(s) could not be mapped:`, Array.from(unmappedBones).sort().join(', '));
         }