From 8954e1d6e1ad5b6e795837718d896501ec145707 Mon Sep 17 00:00:00 2001
From: David Rowe <david@ctrlaltstudio.com>
Date: Wed, 8 Oct 2014 22:11:36 -0700
Subject: [PATCH] Document non-usage of Leap controller's first finger joints

---
 examples/leapHands.js | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/examples/leapHands.js b/examples/leapHands.js
index a322175575..c9c3af0063 100644
--- a/examples/leapHands.js
+++ b/examples/leapHands.js
@@ -23,7 +23,7 @@ var leapHands = (function () {
         fingers,
         NUM_FINGERS = 5,  // 0 = thumb; ...; 4 = pinky
         THUMB = 0,
-        NUM_FINGER_JOINTS = 3,  // 0 = metacarpal(hand)-proximal(finger) joint; ...; 2 = intermediate-distal(tip) joint
+        NUM_FINGER_JOINTS = 3,  // 0 = metacarpal(hand)-proximal(finger) joint; ...; 2 = intermediate-distal joint
         MAX_HAND_INACTIVE_COUNT = 20,
         calibrationStatus,
         UNCALIBRATED = 0,
@@ -226,8 +226,6 @@ var leapHands = (function () {
 
     function setUp() {
 
-        // TODO: Leap Motion controller joint naming doesn't match up with skeleton joint naming; numbers are out by 1.
-
         hands = [
             {
                 jointName: "LeftHand",
@@ -246,6 +244,9 @@ var leapHands = (function () {
             { controller: Controller.createInputController("Spatial", "joint_R_wrist") }
         ];
 
+        // The Leap controller's first joint is the hand-metacarpal joint but this joint's data is not used because it's too
+        // dependent on the model skeleton exactly matching the Leap skeleton; using just the second and subsequent joints 
+        // seems to work better over all.
         fingers = [{}, {}];
         fingers[0] = [
             [