search

scheinerman / Permutations.jl

Permutations class for Julia.
Other
51 stars 14 forks source link

Transpose Matrix #14

Closed dlfivefifty closed 6 years ago

dlfivefifty commented 6 years ago

This ensures Matrix(::Permutation) is a representation, that is, 

a,b,c,d = Permutation([2,1,3,4]),Permutation([1,2,4,3]),
                Permutation([1,3,2,4]),Permutation([2,1,3,4])
@test Matrix(a*b*c*d) == Matrix(a)*Matrix(b)*Matrix(c)*Matrix(d)

Note that now Matrix(a)' * v is the corresponding permutation of the entries.

scheinerman commented 6 years ago

While I wait to hear whether or not there is standard, would you be willing also to update the README to reflect the new definition of Matrix? I'm expecting to accept the change.

dlfivefifty commented 6 years ago

I pushed to this PR an updated README.

scheinerman commented 6 years ago

Thanks. I'm good with this now.