search

gap-packages / wedderga

Wedderburn Decomposition of Group Algebras
https://gap-packages.github.io/wedderga/
GNU General Public License v2.0
3 stars 6 forks source link

wedderga testsuite fails on master when using a minimal set of packages #84

Closed fingolfin closed 2 years ago

fingolfin commented 4 years ago

It looks as if perhaps testinstall.g needs LoadPackage("atlasrep")?

########> Diff in /home/gap/inst/gap-master/pkg/wedderga-4.10.0/tst/wedderga07.tst:72
# Input is:
G:=AtlasGroup("J2");
# Expected output:
<permutation group of size 604800 with 2 generators>
# But found:
Error, Variable: 'AtlasGroup' must have a value
########

Full log is available at https://travis-ci.org/github/gap-infra/gap-docker-pkg-tests-master/jobs/710776137

drallenherman commented 4 years ago

Yes, we kept this atlas group test in the 4.10.0 documentation for demonstration purposes only. So it is not crucial and if it causes a problem it could be taken out.

fingolfin commented 4 years ago

You could just turn it from an <Example> into a <Log> then it won't be included in the tests. Or else just add LoadPackage("atlasrep"); to the start of your tst/testall.g.

olexandr-konovalov commented 4 years ago

I'm for adding LoadPackage("atlasrep"); call.

drallenherman commented 4 years ago

Okay. Go ahead @alex-konovalov let's load the package then. Can you take care of it?

olexandr-konovalov commented 4 years ago

@drallenherman yes, I wil!

olexandr-konovalov commented 4 years ago

Still here e.g. https://travis-ci.org/github/gap-infra/gap-docker-pkg-tests-master/jobs/714790646 - should do in August.