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

Try out another branch #972

Closed
wants to merge 4 commits into from
Closed

Conversation

hulpke
Copy link
Contributor

@hulpke hulpke commented Dec 2, 2016

Just to see what is hapening, this is another branch, freshly branched off master, with the cherry-picked changes. It is only intended to trigger the travis-tests to see whether they also cause an error.

FIX: More lenient in using maxes for factor group perm rep.
Use structure of automorphisms of socle to write down permutations for it.

but careful with primitive library
…pairs.

FIX: First abelian, then primitive to avoid recursion.

FIX: Improved heuristics for particular approaches
@hulpke hulpke added the do not merge PRs which are not yet ready to be merged (e.g. submitted for discussion, or test results) label Dec 2, 2016
@olexandr-konovalov
Copy link
Member

I think that the actuall error is in

########> Diff in /home/travis/build/gap-system/gap/tst/testinstall/opers/Comp\
lementClassesRepresentatives.tst:24
# Input is:
N := Group((6,7,8),(6,7,8,9,10));; ConjugacyClassesSubgroups(G);;
# Expected output:
# But found:
Error, no method found! For debugging hints type ?Recovery from NoMethodFound
The 1st argument is 'fail' which might point to an earlier problem
Error, no 1st choice method found for `SubgroupsOrbitsAndNormalizers' on 3 arg\
\
uments
########

and the 2nd is just caused by using some other N instead, since the previous command failed:

########> Diff in /home/travis/build/gap-system/gap/tst/testinstall/opers/Comp\
lementClassesRepresentatives.tst:25
# Input is:
Set(ComplementClassesRepresentatives(G, N), H -> H^G)=Set([ Group([ (1,2,5)(3,\
4)(6,7), (1,4)(2,5,3)(6,7) ])^G, Group([ (2,5)(7,8), (1,3,5,4)(6,10,9,7) ])^G,\
 Group([ (2,3,4)(6,8,10), (1,2)(3,5,4)(6,10,7)(8,9) ])^G ]);
# Expected output:
true
# But found:
Error, cannot compute complements if both N and G/N are nonsolvable
########

This does not explain why it's here and not in the other tests though.

@hulpke
Copy link
Contributor Author

hulpke commented Dec 2, 2016

This causes the same errors, so no no need to pursue.

@hulpke hulpke closed this Dec 2, 2016
@hulpke hulpke deleted the enhancements branch December 19, 2017 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge PRs which are not yet ready to be merged (e.g. submitted for discussion, or test results)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants