[quake3-bugzilla] [Bug 5199] IQM joint matrices wrong

bugzilla-daemon at icculus.org bugzilla-daemon at icculus.org
Thu Aug 25 23:51:19 EDT 2011


https://bugzilla.icculus.org/show_bug.cgi?id=5199

James Canete <use.less01 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #2949|0                           |1
        is obsolete|                            |

--- Comment #4 from James Canete <use.less01 at gmail.com> 2011-08-25 23:51:16 EDT ---
Created attachment 2951
  --> https://bugzilla.icculus.org/attachment.cgi?id=2951
IQM joint matrices fix v2

Here's a version of the patch that resembles the example code as much as
possible.  It's still comparable in performance, 27 mults in this vs. 36 mults
in the other patch, though I haven't actually run benchmarks or anything. :)

Outside of that, the easiest to read solution would probably involve writing a
proper matrix inversion function, instead of this function which assumes the
matrix is a combination of scales, rotations, and translations.  I haven't
tested this function with non-uniform scales though, so I don't know if it
works in all cases.

Readability probably isn't that important as long as it matches the example
code though, since it's unlikely to change unless the example code does. 
Assuming the example code is correct, anyway.

-- 
Configure bugmail: https://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.


More information about the quake3-bugzilla mailing list