danielrademacher / BruhatDecomposition

Other
2 stars 0 forks source link

PermutationMonomialMatrixNC is not defined #1

Closed FriedrichRober closed 3 years ago

FriedrichRober commented 3 years ago

Calling the function BruhatDecompositionSLNC results in an error, since PermutationMonomialMatrixNC is not defined.

gap> mat := [ [ Z(5)^2, Z(5)^0, Z(5)^2 ], [ Z(5)^3, 0*Z(5), 0*Z(5) ],[ 0*Z(5), Z(5)^2, Z(5)^2 ] ];;#!
gap> L := BruhatDecompositionSLNC(LGOStandardGensSL(3,5), mat);
#I  returns an SLP to generate u1, u2, p_sign, diag

#I  Memory Usage is: 22 memory slots in UnipotentDecomposition()

Error, PermutationMonomialMatrixNC: function is not yet defined in
  PermutationMonomialMatrixNC( mat 
 ) at /Users/friedrich/.gap/pkg/BruhatDecomposition2/gap/BruhatDecompositionSL.gi:3008 called from 
PermSLPNC( stdgens, pm, slp 
 ) at /Users/friedrich/.gap/pkg/BruhatDecomposition2/gap/BruhatDecompositionSL.gi:3594 called from
<function "BruhatDecompositionSLNC">( <arguments> )
 called from read-eval loop at *stdin*:5
danielrademacher commented 3 years ago

Thanks @FriedrichRober .

The function and associated tests have been added. Please let me know if there are any further errors.