gap-system / gap

Main development repository for GAP - Groups, Algorithms, Programming, a System for Computational Discrete Algebra
https://www.gap-system.org
GNU General Public License v2.0
810 stars 160 forks source link

Error when calling Irr for permutation group #1515

Closed ThomasBreuer closed 7 years ago

ThomasBreuer commented 7 years ago

The following happens in GAP 4.8.7 as well as in the current master branch, without packages.

gap> GAPInfo.Version;
"4.8.7"
gap> s10:= SymmetricGroup( 10 );;
gap> Irr( Stabilizer( s10, [ 1 .. 3 ], OnSets ) );; # this works
gap> Irr( Stabilizer( s10, [ 2 .. 4 ], OnSets ) );; # this does not work
Error, List Element: <list>[2] must have an assigned value in
  img := opr( img, S.transimages[bimg[1]] )
 ; at /users/sam/gap/gap4r8/lib/ghomperm.gi:108 called from 
ImageSiftedBaseImage( S, OnTuples( BaseStabChain( S ), elm ), S.idimage, 
 OnRight ) at /users/sam/gap/gap4r8/lib/ghomperm.gi:1008 called from
ImagesRepresentative( InverseGeneralMapping( hom ), elm 
 ) at /users/sam/gap/gap4r8/lib/ghomperm.gi:1101 called from
PreImagesRepresentative( hom, conj 
 ) at /users/sam/gap/gap4r8/lib/clashom.gi:2786 called from
TFCanonicalClassRepresentative( D.group, [ el ] 
 ) at /users/sam/gap/gap4r8/lib/ctblperm.gi:62 called from
D.identification( D, el 
 ) at /users/sam/gap/gap4r8/lib/ctblgrp.gi:1647 called from
...  at line 4 of *stdin*
you can 'return;' after assigning a value
brk> 
markuspf commented 7 years ago

This is fixed in #1390 by @hulpke

Maybe we need to backport that fix (https://github.com/gap-system/gap/pull/1390/commits/29e1286687e856882af6f020886a5c58334f5fe1)?

olexandr-konovalov commented 7 years ago

@ThomasBreuer thank you for reporting this - I've backported the fix from the master branch in https://github.com/gap-system/gap/pull/1566 so this will be fixed in GAP 4.8.8 coming shortly.