Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ConjugacyClassesSubgroups fails for SmallGroup(1344,11293) #4854

Closed
Stefan-Kohl opened this issue Apr 7, 2022 · 5 comments · Fixed by #5033
Closed

ConjugacyClassesSubgroups fails for SmallGroup(1344,11293) #4854

Stefan-Kohl opened this issue Apr 7, 2022 · 5 comments · Fixed by #5033
Labels
kind: bug: unexpected error Issues describing bugs in which computation unexpectedly encounters an error, and PRs fixing them kind: bug Issues describing general bugs, and PRs fixing them

Comments

@Stefan-Kohl
Copy link
Member

Stefan-Kohl commented Apr 7, 2022

Observed behaviour

gap> ConjugacyClassesSubgroups(SmallGroup(1344,11293));
Error, List Element: <list>[17] must have an assigned value in
  depth > maxdepth at /user/GAP/gap-4.11.1/lib/pcgsperm.gi:814 called from 
ExponentsOfPcElementPermGroup( pcgs, g, 1, Maximum( poss ), 'e' 
 ) at /user/GAP/gap-4.11.1/lib/pcgsperm.gi:1262 called from
ExponentsOfPcElement( ParentPcgs( pcgs ), elm, pcgs!.depthsInParent 
 ) at /user/GAP/gap-4.11.1/lib/pcgsind.gi:1472 called from
ExponentsOfPcElement( ocr.modulePcgs, x 
 ) at /user/GAP/gap-4.11.1/lib/onecohom.gi:264 called from
ocr.moduleMap( n ) at /user/GAP/gap-4.11.1/lib/onecohom.gi:1037 called from
OCEquationVector( ocr, rels[i] ) at /user/GAP/gap-4.11.1/lib/onecohom.gi:1247 called from
...  at *stdin*:1
type 'quit;' to quit to outer loop

Copy and paste GAP banner (to tell us about your setup)

 ┌───────┐   GAP 4.11.1 of 2021-03-02
 │  GAP  │   https://www.gap-system.org
 └───────┘   Architecture: x86_64-pc-linux-gnu-default64-kv7
 Configuration:  gmp 5.1.3, GASMAN
 Loading the library and packages ...
 Packages:   GAPDoc 1.6.4, PrimGrp 3.4.1, SmallGrp 1.4.2, TransGrp 3.0
 Try '??help' for help. See also '?copyright', '?cite' and '?authors'
@fingolfin
Copy link
Member

Works fine in master

@hulpke
Copy link
Contributor

hulpke commented Apr 9, 2022

Since this is already fixed in master, I close

@hulpke hulpke closed this as completed Apr 9, 2022
@stertooy
Copy link
Contributor

I don't think this is fixed, I encountered the same error with a recent build of the master branch. However, it only happens when GAP is ran with the -A flag, in particular when TomLib is not loaded.

hulpke added a commit to hulpke/gap that referenced this issue Jul 27, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a slightly
different format than required.
If yomlib is loded its a mote issue, and thus does not merit investigation.

This closes gap-system#4854
@fingolfin fingolfin reopened this Jul 28, 2022
@fingolfin fingolfin added kind: bug Issues describing general bugs, and PRs fixing them kind: bug: unexpected error Issues describing bugs in which computation unexpectedly encounters an error, and PRs fixing them and removed kind: bug : already fixed in master labels Jul 28, 2022
@fingolfin
Copy link
Member

I can confirm this:

$ ./gap -A
 ┌───────┐   GAP 4.12dev-1388-g3ef5404-dirty built on 2022-07-28 15:57:10+0200
 │  GAP  │   https://www.gap-system.org
 └───────┘   Architecture: aarch64-apple-darwin21-default64-kv8
 Configuration:  gmp 6.2.1, GASMAN, readline
 Loading the library and packages ...
 Packages:   GAPDoc 1.6.5, PrimGrp 3.4.2, SmallGrp 1.5, TransGrp 3.6.2
 Try '??help' for help. See also '?copyright', '?cite' and '?authors'
gap> ConjugacyClassesSubgroups(SmallGroup(1344,11293));
Error, List Element: <list>[17] must have an assigned value in
  depth > maxdepth at GAPROOT/lib/pcgsperm.gi:816 called from
ExponentsOfPcElementPermGroup( pcgs, g, 1, Maximum( poss ), 'e'
 ) at GAPROOT/lib/pcgsperm.gi:1264 called from
ExponentsOfPcElement( ParentPcgs( pcgs ), elm, pcgs!.depthsInParent
 ) at GAPROOT/lib/pcgsind.gi:1475 called from
ExponentsOfPcElement( ocr.modulePcgs, x
 ) at GAPROOT/lib/onecohom.gi:264 called from
ocr.moduleMap( n ) at GAPROOT/lib/onecohom.gi:1037 called from
OCEquationVector( ocr, rels[i]
 ) at GAPROOT/lib/onecohom.gi:1247 called from
...  at *stdin*:1
type 'quit;' to quit to outer loop
brk>

@fingolfin
Copy link
Member

With assertions turned up a bit:

gap> SetAssertionLevel(1);
gap> ConjugacyClassesSubgroups(SmallGroup(1344,11293));
Error, Assertion failure in
  KernelOfMultiplicativeGeneralMapping( fphom ) = nts[j] at GAPROOT/lib/grplatt.gi:1192 called from
LatticeSubgroups( G ) at GAPROOT/lib/grplatt.gi:208 called from
<function "ConjugacyClassesSubgroups group">( <arguments> )
 called from read-eval loop at *stdin*:2
you may 'return;'
brk>

hulpke added a commit to hulpke/gap that referenced this issue Aug 9, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a slightly
different format than required.
If yomlib is loded its a mote issue, and thus does not merit investigation.

This closes gap-system#4854
hulpke added a commit to hulpke/gap that referenced this issue Aug 9, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a slightly
different format than required.
If yomlib is loded its a mote issue, and thus does not merit investigation.

This closes gap-system#4854
hulpke added a commit to hulpke/gap that referenced this issue Aug 23, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a slightly
different format than required.
If yomlib is loded its a mote issue, and thus does not merit investigation.

This closes gap-system#4854
hulpke added a commit to hulpke/gap that referenced this issue Sep 3, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a slightly
different format than required.
If yomlib is loded its a mote issue, and thus does not merit investigation.

This closes gap-system#4854
hulpke added a commit to hulpke/gap that referenced this issue Sep 5, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a slightly
different format than required.
If yomlib is loded its a mote issue, and thus does not merit investigation.

This closes gap-system#4854
hulpke added a commit to hulpke/gap that referenced this issue Sep 5, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a slightly
different format than required.
If yomlib is loded its a mote issue, and thus does not merit investigation.

This closes gap-system#4854
hulpke added a commit to hulpke/gap that referenced this issue Sep 5, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a slightly
different format than required.
If yomlib is loded its a mote issue, and thus does not merit investigation.

This closes gap-system#4854
hulpke added a commit to hulpke/gap that referenced this issue Sep 5, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a slightly
different format than required.
If yomlib is loded its a mote issue, and thus does not merit investigation.

This closes gap-system#4854
fingolfin pushed a commit to fingolfin/gap that referenced this issue Sep 5, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a
slightly different format than required. This resulted in some commands,
e.g. `ConjugacyClassesSubgroups(SmallGroup(1344,11293))`, to enter a
break loop.

If tomlib is loded it's a mote issue, and thus does not merit further
investigation.

This resolves gap-system#4854
fingolfin pushed a commit that referenced this issue Sep 7, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a
slightly different format than required. This resulted in some commands,
e.g. `ConjugacyClassesSubgroups(SmallGroup(1344,11293))`, to enter a
break loop.

If tomlib is loded it's a mote issue, and thus does not merit further
investigation.

This resolves #4854
fingolfin pushed a commit to fingolfin/gap that referenced this issue Sep 9, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a
slightly different format than required. This resulted in some commands,
e.g. `ConjugacyClassesSubgroups(SmallGroup(1344,11293))`, to enter a
break loop.

If tomlib is loded it's a mote issue, and thus does not merit further
investigation.

This resolves gap-system#4854
fingolfin pushed a commit that referenced this issue Sep 9, 2022
This disables an attempt to use cyclic extension instead of lifting for
lattices of smallish groups. The issue is that CE returns data in a
slightly different format than required. This resulted in some commands,
e.g. `ConjugacyClassesSubgroups(SmallGroup(1344,11293))`, to enter a
break loop.

If tomlib is loded it's a mote issue, and thus does not merit further
investigation.

This resolves #4854
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: bug: unexpected error Issues describing bugs in which computation unexpectedly encounters an error, and PRs fixing them kind: bug Issues describing general bugs, and PRs fixing them
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants