setFromTwoVectors AFAIK is (almost always?) used á la mode de AngleAxis or Identity.
As such it seems that it should exist as either a constructor or static function.
good point to add a FromTwoVectors static function. For the ctor, I'm not fan because it would be quite confusing.
Created attachment 260 [details]
Patch to add FromTwoVectors static function
Static function which calls the non-static.
Copy pasted the unit test, and ran unit test.
thanks for the patch, I committed a variant: this function has to be part of Quaternion, not QuaternionBase.
date: 2012-03-26 18:30:04
summary: fix bug 439: add Quaternion::FromTwoVectors() static constructor
Thanks for the quick action! : )
-- GitLab Migration Automatic Message --
This bug has been migrated to gitlab.com's GitLab instance and has been closed from further activity.
You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.com/libeigen/eigen/issues/439.