Address feedback on PR, in particular property comments, variable names

Author: aymanhab

OpenSim/Tools/InverseKinematicsTool.cpp

@@ -162,11 +162,11 @@ bool InverseKinematicsTool::run()
         // specified then use time from marker reference.
         // Adjust the time range for the tool if the provided range exceeds
         // that of the marker data.
-        SimTK::Vec2 markersValidTimRange = markersReference.getValidTimeRange();
-        double start_time = (markersValidTimRange[0] > get_time_range(0)) ?
-            markersValidTimRange[0] : get_time_range(0);
-        double final_time = (markersValidTimRange[1] < get_time_range(1)) ?
-            markersValidTimRange[1] : get_time_range(1);
+        SimTK::Vec2 markersValidTimeRange = markersReference.getValidTimeRange();
+        double start_time = (markersValidTimeRange[0] > get_time_range(0)) ?
+            markersValidTimeRange[0] : get_time_range(0);
+        double final_time = (markersValidTimeRange[1] < get_time_range(1)) ?
+            markersValidTimeRange[1] : get_time_range(1);
 
         SimTK_ASSERT2_ALWAYS(final_time >= start_time,
             "InverseKinematicsTool final time (%f) is before start time (%f).",

OpenSim/Tools/InverseKinematicsTool.h

@@ -77,15 +77,17 @@ OpenSim_DECLARE_CONCRETE_OBJECT(InverseKinematicsTool, Tool);
 
     OpenSim_DECLARE_PROPERTY(accuracy, double,
             "The accuracy of the solution in absolute terms, i.e. the number of "
-            "significant digits to which the solution can be trusted. Default 1e-6.");
+            "significant digits to which the solution can be trusted. Default 1e-5.");
 
     OpenSim_DECLARE_UNNAMED_PROPERTY(
             IKTaskSet, 
             "Markers and coordinates to be considered (tasks) and their weightings. "
             "The sum of weighted-squared task errors composes the cost function.");
 
     OpenSim_DECLARE_PROPERTY(marker_file, std::string,
-            "Name/path to a .trc or .sto file of type Vec3 of marker data.");
+            "TRC file (.trc) containing the time history of observations of marker "
+            "positions obtained during a motion capture experiment. Markers in this "
+            "file that have a corresponding task and model marker are included."");
 
 
     OpenSim_DECLARE_PROPERTY(coordinate_file, std::string,

OpenSim/Tools/Tool.h

@@ -73,7 +73,8 @@ OpenSim_DECLARE_ABSTRACT_OBJECT(Tool, Object);
     /**
     * Default constructor.
     */
-    Tool() { constructProperties();
+    Tool() { 
+        constructProperties();
     };
 
     /**