AnimatedTransform doesn't handle sign flips in scale matrices #41

mmp opened this Issue Oct 19, 2015 · 0 comments


None yet

1 participant

mmp commented Oct 19, 2015

If the two transformations given to it have a mirroring between them (i.e. differing signs in some components), then the motion bounds it computes are incorrect; this can be seen by removing the three std::abs calls in t = t * Scale(std::abs(r()), std::abs(r()), std::abs(r())); in src/tests/animatedtransform.cpp.

This is admittedly not a common case, but would be nice to handle correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment