laurentbartholdi
commented
7 years ago Thanks!
This unearthed a bug in GAP in that it sometimes modifies an argument secretly, see Issue https://github.com/gap-system/gap/issues/1179 which I just filed.
On Tue, 7 Mar 2017 at 11:21 Alexander Konovalov notifications@github.com wrote:
The following happens while testing the package with the GAP stable-4.8 branch:
============================OUTPUT START==============================
I RunPackageTests("fr", "2.4.1", "tst/testall.g", auto);
��������������������������� GAP unknown of 2017-03-06 13:07:58 (GMT) ��� GAP ��� https://www.gap-system.org ��������������������������� Architecture: x86_64-pc-linux-gnu-gcc-default64 Libs used: gmp, readline Loaded workspace: wsp.g Components: trans 1.0, prim 2.1, small 1.0, id 1.0 Packages: ACE 5.2, AClib 1.2, Alnuth 3.0.0, AtlasRep 1.5.1, AutPGrp 1.8, Browse 1.8.6, CRISP 1.4.4, Cryst 4.1.12, CrystCat 1.1.6, CTblLib 1.2.2, FactInt 1.5.4, FGA 1.3.1, FR 2.4.1, GAPDoc 1.5.1, GBNP 1.0.3, IO 4.4.6, IRREDSOL 1.3.1, LAGUNA 3.7.0, lpres 0.4.1, nq 2.5.3, Polenta 1.3.7, Polycyclic 2.11, RadiRoot 2.7, ResClasses 4.6.0, Sophus 1.23, SpinSym 1.5, TomLib 1.2.6, Utils 0.46 Try '??help' for help. See also '?copyright', '?cite' and '?authors'
I ReadTest is no longer supported. Please use more robust and flexible
I Test. For backwards compatibility, ReadTest(
) is replaced I by Test(
, rec( compareFunction := "uptowhitespace" )) fr:chapter 12 GAP4stones: 300000000
I ReadTest is no longer supported. Please use more robust and flexible
I Test. For backwards compatibility, ReadTest(
) is replaced I by Test(
, rec( compareFunction := "uptowhitespace" )) fr:chapter 3 GAP4stones: 1020408
I ReadTest is no longer supported. Please use more robust and flexible
I Test. For backwards compatibility, ReadTest(
) is replaced I by Test(
, rec( compareFunction := "uptowhitespace" )) fr:chapter 4 GAP4stones: 997804
I ReadTest is no longer supported. Please use more robust and flexible
I Test. For backwards compatibility, ReadTest(
) is replaced I by Test(
, rec( compareFunction := "uptowhitespace" )) ########> Diff in /data/gap-jenkins/workspace/GAP-pkg-update-stable-quicktest/\ GAPCOPTS/64build/GAPGMP/gmp/GAPTARGET/packages/label/kovacs/GAP-pkg-update-sta\ ble-snapshot/pkg/FR-2.4.1/tst/chapter-5-a.tst, line 1276:
Input is:
a := mealyel[1][1][2];
Expected output:
<Mealy element on alphabet [ 1 .. 2 ] with 2 states>
But found:
<Mealy element on alphabet [ 1, 2 ] with 2 states> ######## fr:chapter 5 (1/2) GAP4stones: 637755
I ReadTest is no longer supported. Please use more robust and flexible
I Test. For backwards compatibility, ReadTest(
) is replaced I by Test(
, rec( compareFunction := "uptowhitespace" )) ########> Diff in /data/gap-jenkins/workspace/GAP-pkg-update-stable-quicktest/\ GAPCOPTS/64build/GAPGMP/gmp/GAPTARGET/packages/label/kovacs/GAP-pkg-update-sta\ ble-snapshot/pkg/FR-2.4.1/tst/chapter-5-b.tst, line 111:
Input is:
AlphabetInvolution(GammaPQMachine(3,5));
Expected output:
[ 6, 5, 4, 3, 2, 1 ]
But found:
I MakeReadWriteGlobal: QUATERNIONBASIS@fr already read-write
Error, Variable: 'QUATERNIONBASIS@FR' is read only ######## ########> Diff in /data/gap-jenkins/workspace/GAP-pkg-update-stable-quicktest/\ GAPCOPTS/64build/GAPGMP/gmp/GAPTARGET/packages/label/kovacs/GAP-pkg-update-sta\ ble-snapshot/pkg/FR-2.4.1/tst/chapter-5-b.tst, line 113:
Input is:
AlphabetInvolution(GammaPQMachine(5,7));
Expected output:
[ 8, 7, 6, 5, 4, 3, 2, 1 ]
But found:
I MakeReadWriteGlobal: QUATERNIONBASIS@fr already read-write
Error, Variable: 'QUATERNIONBASIS@FR' is read only ######## ########> Diff in /data/gap-jenkins/workspace/GAP-pkg-update-stable-quicktest/\ GAPCOPTS/64build/GAPGMP/gmp/GAPTARGET/packages/label/kovacs/GAP-pkg-update-sta\ ble-snapshot/pkg/FR-2.4.1/tst/chapter-5-b.tst, line 380:
Input is:
perm := Random(SymmetricGroup(5));
Expected output:
(2,4,3)
But found:
(1,2,5,4,3) ######## fr:chapter 5 (2/2) GAP4stones: 520833
I RunPackageTests("fr", "2.4.1", "tst/testall.g", auto): runtime 15177
============================OUTPUT END================================
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/gap-packages/fr/issues/19, or mute the thread https://github.com/notifications/unsubscribe-auth/ACIIUBEYPY8qyxu4sFGHCamNG9LKgAKvks5rjS_CgaJpZM4MVSrg .
-- Laurent Bartholdi DMA, École Normale Supérieure, 45 rue d'Ulm, 75005 Paris. +33 14432 2060 Mathematisches Institut, Universität Göttingen, Bunsenstrasse 3-5, D-37073 Göttingen. +49 551 39 7826
olexandr-konovalov
The following happens while testing the package with the GAP stable-4.8 branch: