HeavenWu / slimdx

Automatically exported from code.google.com/p/slimdx
MIT License
0 stars 0 forks source link

Quaternion multiplication (again) #657

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Hi SlimDX team!

This is related to the closed Issue 534.
We recently update to SlimDX Feb. 2010.
After that our Trackball Camera implementation did not work correctly. 
I found out that the Quaternion multiplication swapped the left and right
parameter as proposed in Issue 534.
Since our code already worked with MS Managed DirectX and the call to
D3DXQuaternionMultiply() gave us the save result as the SlimDX version
before revision 1199, I would assume that the current implementation of the
quaternion multiplication is wrong. 

Best Regards
Karol

Original issue reported on code.google.com by k...@ventuz.com on 21 Apr 2010 at 9:34

GoogleCodeExporter commented 8 years ago
Unfortunately we got torn between the way XNA does it and the way D3DX does it. 
As
much as I hate to change it again, most everything else we do matches D3DX so 
I'll
switch it back.

Original comment by Mike.Popoloski on 29 Apr 2010 at 11:49

GoogleCodeExporter commented 8 years ago
Dear Mike, 

First of all, congrats to you, Promit, and all the people involved in SlimDX.

We have been also affected by this issue in our physics engine. Is it official 
that 
you will switch back to the previos quaternion multiplication algorithm in the 
next 
release? (Ive checked the current source code and it´s still the XNA one).

Thanks a lot and best regards, 

Inaki Ayucar
(http://graphicdna.blogspot.com)

Original comment by iayucart...@gmail.com on 26 May 2010 at 3:47

GoogleCodeExporter commented 8 years ago
Yes, it's official. We will switch back for the June release. I will update this
issue once I've committed the change.

Original comment by Mike.Popoloski on 26 May 2010 at 4:55

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r1594.

Original comment by Mike.Popoloski on 29 May 2010 at 2:12