-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rotation matrices don't match reference solution? #178
Comments
Update I dug a little bit deeper. I compared the loop solution with an updated modulo function that takes into account 0-start-indexing and a written out matrix from p.89 "M. A. Slawinski, Seismic waves and rays in elastic media. Amsterdam: Pergamon, 2003." It does not matter which rotation I make, the looped and fully typed rotation matrix are the same and as such don't match the reference solution in the same places. I also compared to other reference solutions, e.g. this one. Something is not correct and I am not able to find the error.
|
I solved it. The problem was about active and passive rotation. I used the same angle for both the reference and the computed rotation. It appears that for the reference the sign of the angle had to be reversed. See also the corresponding physics stackexchange post |
Description
The rotation matrix given in http://solidmechanics.org/Text/Chapter3_2/Chapter3_2.php under section 3.2.11 is not the same as what I get from the package. It might be an issue with clockwise and counterclockwise rotation, as the numbers are the same, the signs are not always.
As far as I understand it, a counterclockwise rotation should be positive about the rotation axis according to the right-hand rule. This is also what's shown in his book and the figure under 3.2.11 on the webpage.
I know this is a version under development, but I checked your code and did not find any error.
What I Did
Computed the two rotation matrices as below. They don't match.
Output:
The text was updated successfully, but these errors were encountered: