Skip to content

Commit b3ad3ef

Browse files
authored
FIX: RepresentativesPerfectSubgroups (#6160)
Do not just pick normal subgroups that are normal in minimal ones on top, but their full G-orbits, with actions. This seems to avoid representative selection issues.
1 parent 079309f commit b3ad3ef

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

lib/grplatt.gi

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -992,6 +992,7 @@ InstallGlobalFunction(LatticeViaRadical,function(arg)
992992
if Length(mpcgs)>0 then
993993
gf:=GF(RelativeOrders(mpcgs)[1]);
994994
if select=IsPerfectGroup then
995+
995996
# the only normal subgroups are those that are normal under any
996997
# subgroup so far.
997998

@@ -1008,7 +1009,13 @@ InstallGlobalFunction(LatticeViaRadical,function(arg)
10081009
od;
10091010
SortBy(nts,Size); # increasing order
10101011
# by setting up `act' as fail, we force a different selection later
1011-
act:=[nts,fail];
1012+
#act:=[nts,fail];
1013+
1014+
# there is an issue in selecting representatives. Form G-orbits to
1015+
# remain in usual case.
1016+
nts:=Union(Orbits(G,nts));
1017+
act:=ActionHomomorphism(G,nts,"surjective");
1018+
act:=[nts,act];
10121019

10131020
elif select=IsNonabelianSimpleGroup then
10141021
# simple -> no extensions, only the trivial subgroup is valid.
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
# missing subgroups when finding perfect subgroups
2+
# See <https://github.com/gap-system/gap/issues/6157>.
3+
gap> G:=WreathProduct(Group((1,2)), SymmetricGroup(6));;
4+
gap> Size(ConjugacyClassesPerfectSubgroups(G));
5+
8

0 commit comments

Comments
 (0)