Skip to content

Commit

Permalink
FIX: IntermediateGroup will not exit quietly if index is large
Browse files Browse the repository at this point in the history
  • Loading branch information
hulpke committed Feb 7, 2021
1 parent 177da5c commit ce446ff
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 1 deletion.
11 changes: 10 additions & 1 deletion lib/csetgrp.gi
Original file line number Diff line number Diff line change
Expand Up @@ -330,6 +330,12 @@ local o,b,img,G1,c,m,mt,hardlimit,gens,t,k,intersize;
return fail;
fi;

# try the `AscendingChain` mechanism
c:=AscendingChain(G,U:refineIndex:=QuoInt(IndexNC(G,U),2));
if Length(c)>2 then
return First(c,x->Size(x)>Size(U));
fi;

# old code -- obsolete

c:=ValueOption("refineChainActionLimit");
Expand All @@ -339,7 +345,10 @@ local o,b,img,G1,c,m,mt,hardlimit,gens,t,k,intersize;
hardlimit:=1000000;
fi;

if Index(G,U)>hardlimit then return fail;fi;
if Index(G,U)>hardlimit then
Info(InfoWarning,1,
"will have to use permutation action of degree bigger than ", hardlimit);
fi;

if IsPermGroup(G) and Length(GeneratorsOfGroup(G))>3 then
G1:=Group(SmallGeneratingSet(G));
Expand Down
5 changes: 5 additions & 0 deletions tst/testbugfix/2021-02-07-IntermediateGroup.tst
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
# IntermediateGroup in large index, reported in Forum (Breuer/Anvita) on 2/7/21
gap> L:=PSL(2,7^3);;
gap> S:=SylowSubgroup(L,2);;
gap> IntermediateGroup(L,S);
<permutation group of size 344 with 4 generators>

0 comments on commit ce446ff

Please sign in to comment.