Skip to content

Commit 81acb0f

Browse files
authored
Don't transfer nice monos when conjugating (#6207)
We tried this but then decide against it, because we determined that often you conjugate precisely so that you get a nicer description of a group. But by accident the code implementing it got merged anyway.
1 parent f7a40f7 commit 81acb0f

File tree

1 file changed

+1
-12
lines changed

1 file changed

+1
-12
lines changed

lib/grpmat.gi

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1274,7 +1274,7 @@ InstallMethod( InvariantBilinearForm,
12741274
InstallMethod( ConjugateGroup, "<G>, <g>", IsCollsElms,
12751275
[ IsMatrixGroup, IsMultiplicativeElementWithInverse ],
12761276
function( G, g )
1277-
local H, m, ginv, nice, conj;
1277+
local H, m, ginv;
12781278

12791279
H := GroupByGenerators( OnTuples( GeneratorsOfGroup( G ), g ), One( G ) );
12801280
UseIsomorphismRelation( G, H );
@@ -1317,16 +1317,5 @@ InstallMethod( ConjugateGroup, "<G>, <g>", IsCollsElms,
13171317
SetIsFullSubgroupGLorSLRespectingQuadraticForm( H, true );
13181318
fi;
13191319
fi;
1320-
if HasNiceMonomorphism( G ) then
1321-
nice := NiceMonomorphism( G );
1322-
if not IsBound( ginv ) then
1323-
ginv := g^-1;
1324-
fi;
1325-
conj := GroupHomomorphismByFunction( H, G, y -> g*y*ginv, x -> ginv*x*g );
1326-
SetNiceMonomorphism( H, conj * nice );
1327-
if HasNiceObject( G ) then
1328-
SetNiceObject( H, NiceObject( G ) );
1329-
fi;
1330-
fi;
13311320
return H;
13321321
end );

0 commit comments

Comments
 (0)