omair18 / matlab-toolboxes-robotics-vision

Automatically exported from code.google.com/p/matlab-toolboxes-robotics-vision
0 stars 0 forks source link

inertia matrix changes with different base rotations #25

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. create robot model (for example stanford manipulator or baxter robot model 
from 
https://groups.google.com/a/rethinkrobotics.com/forum/#!mydiscussions/brr-users/
5X1-6w-Ja1I)
mdl_stanford 
2. calculate inertia matrix without base transformation
stanf.base = eye(4);
M1 = stanf.inertia(qz)

3. calculate inertia matrix with abitrary different base transformation 
stanf.base = trotz(pi/2)*transl([1;2;3]);
M2 = stanf.inertia(qz)

What is the expected output?
Expected: The inertia matrix should stay the same
What do you see instead?
Output: Inertia matrix changes with base transformation matrix
M1-M2

What version of the product are you using? On what operating system?
Robotics Toolbox 9.9; Linux 64 bit; Should not matter here

Please provide any additional information below.

In rne_dh.m, Line 122 
(https://code.google.com/p/matlab-toolboxes-robotics-vision/source/browse/matlab
/robot/trunk/@SerialLink/rne_dh.m?r=751#112), the base transformation is added 
to the vector from the first joint frame to the second joint frame (pstar). By 
doing this, the lever arm is not calculated for the first joint but for the 
point [0;0;0]. By removing this line, the problem seems solved.

Please contact me, if this is not a bug and I missed something
Regards,

Moritz Schappler,
schappler@irt.uni-hannover.de
Research Assistant at the Institute for Automatic Control, Leibniz Universität 
Hannover, Germany

Original issue reported on code.google.com by baxter....@gmail.com on 28 Jul 2014 at 1:56

GoogleCodeExporter commented 9 years ago
I inserted a link to the wrong revision. Here is the link to the current 
revision
https://code.google.com/p/matlab-toolboxes-robotics-vision/source/browse/matlab/
robot/trunk/%40SerialLink/rne_dh.m#120

Original comment by baxter....@gmail.com on 29 Jul 2014 at 6:12