Skip to content

Commit

Permalink
FIX: DirectSumMat builds field extensions if needed
Browse files Browse the repository at this point in the history
  • Loading branch information
hulpke authored and fingolfin committed Sep 9, 2022
1 parent d11ab96 commit f13d1ed
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/matrix.gi
Original file line number Diff line number Diff line change
Expand Up @@ -4052,7 +4052,7 @@ InstallGlobalFunction( DirectSumMat, function (arg)
if IsSubset(c,F) then
F:=c;
else
Error("build field/ring");
F:=Field(Concatenation(GeneratorsOfField(F),GeneratorsOfField(c)));
fi;
fi;
od;
Expand Down
4 changes: 4 additions & 0 deletions tst/teststandard/permgrp.tst
Original file line number Diff line number Diff line change
Expand Up @@ -136,5 +136,9 @@ false
gap> IsConjugatorAutomorphism(hom);
true

# classes over larger field extension
gap> Length(ConjugacyClasses(PSL(2,64)));
65

#
gap> STOP_TEST( "permgrp.tst", 1);

0 comments on commit f13d1ed

Please sign in to comment.