You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
# Not really sure what this distribution is, but it's also not clear what
Not really sure what this distribution is, but it's also not clear what it should be! rand(RotXY) is invariant to pre-rotations by a RotX and post-rotations by a RotY...
The text was updated successfully, but these errors were encountered:
Hmm. I agree with the status-quo for the first and last, because they are the right measure spaces for 2D and 3D rotations respectively, but I'm not sure what space / measure you should use to define RotXY. The current definition doesn't seem like the one it should be, but I have no idea.
As you say, we could just document it (until someone has a better idea of what it should be).
On current
master
branch:rand(RotX)
is same asRotX(2π*rand())
.rand(RotXY)
is same asRotXY(2π*rand(), 2π*rand())
.rand(RotXYZ)
is not same asRotXYZ(2π*rand(), 2π*rand(), 2π*rand())
.rand(RotXYZ)
is same asRotXYZ(rand(UnitQuaternion))
.This makes confusing, so we have two choices:
rand(RotXYZ)
byRotXYZ(2π*rand(), 2π*rand(), 2π*rand())
.I guess the first choice would be prefered.
cf.
Rotations.jl/src/euler_types.jl
Line 266 in 38b208b
The text was updated successfully, but these errors were encountered: