Summary: | Provide static Quaternion method for uniform random rotation | ||||||
---|---|---|---|---|---|---|---|
Product: | Eigen | Reporter: | edx <eu> | ||||
Component: | Geometry | Assignee: | Nobody <eigen.nobody> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Feature Request | CC: | chtz, gael.guennebaud, hauke.heibel, jacob.benoit.1, josephmirabel | ||||
Priority: | Low | Keywords: | JuniorJob | ||||
Version: | unspecified | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
edx
2014-06-06 10:43:22 UTC
Please provide a clean patch with documentation and its respective unit test (added in test/geo_quaternion.cpp) so that it can be integrated quickly. For "UnitRandom" you can use internal::random<Scalar>(0,1); Thanks. Created attachment 669 [details]
Add uniformly random quaternion
Would this fit the Eigen coding standards ?
Sorry for the delay, but it's now in devel branch: https://bitbucket.org/eigen/eigen/commits/423d3da0e233/ Summary: Bug 823: add static method to Quaternion for uniform random rotations. https://bitbucket.org/eigen/eigen/commits/5d78b569eac3/ Summary: Fix coding practice in Quaternion::UniformRandom https://bitbucket.org/eigen/eigen/commits/20c444245e5c/ Summary: Rename UniformRandom to UnitRandom. -- 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/823. |