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

Failures in GAP master branch #632

Closed
olexandr-konovalov opened this issue Dec 15, 2019 · 11 comments
Closed

Failures in GAP master branch #632

olexandr-konovalov opened this issue Dec 15, 2019 · 11 comments
Assignees

Comments

@olexandr-konovalov
Copy link
Contributor

https://travis-ci.org/gap-infra/gap-docker-pkg-tests-master/jobs/624910497 shows this:

testing: /home/gap/inst/gap-master/pkg/semigroups-3.2.0/tst/standard/attr.tst
######## > Diff in:
/home/gap/inst/gap-master/pkg/semigroups-3.2.0/tst/standard/attr.tst:1210
# Input is:
S := Monoid(Matrix(GF(2 ^ 2),
[[Z(2 ^ 2), 0 * Z(2), 0 * Z(2), 0 * Z(2), 0 * Z(2), 0 * Z(2)],
 [Z(2 ^ 2), 0 * Z(2), 0 * Z(2), 0 * Z(2), 0 * Z(2), 0 * Z(2)],
 [0 * Z(2), Z(2) ^ 0, 0 * Z(2), 0 * Z(2), 0 * Z(2), 0 * Z(2)],
 [0 * Z(2), 0 * Z(2), Z(2) ^ 0, 0 * Z(2), 0 * Z(2), 0 * Z(2)],
 [0 * Z(2), 0 * Z(2), 0 * Z(2), Z(2) ^ 0, 0 * Z(2), 0 * Z(2)],
 [0 * Z(2), 0 * Z(2), 0 * Z(2), 0 * Z(2), 0 * Z(2), 0 * Z(2)]]));;
# Expected output:
# But found:
Error, no method found! For debugging hints type ?Recovery from NoMethodFound
Error, no 2nd choice method found for `OneMutable' on 1 arguments
########
@wilfwilson
Copy link
Collaborator

Thanks. Before looking into this further, I’ll note that Semigroups 3.2.1 is the most recent version (not 3.2.0, as you are testing there), so it’s possible that the problem is fixed already. If the problems still exist with Semigroups 3.2.1, then we’ll definitely look into it properly 🙂

@olexandr-konovalov
Copy link
Contributor Author

Ok - a new test is now running at https://travis-ci.org/gap-infra/gap-docker-pkg-tests-master/builds/625265651, it has Semigroups 3.2.1. Let's watch it.

@olexandr-konovalov
Copy link
Contributor Author

@james-d-mitchell
Copy link
Collaborator

Hmm interesting, I will try to investigate this later today.

@james-d-mitchell
Copy link
Collaborator

@alex-konovalov I think I've fixed one of the issues, I'm not sure about the other one. Once PR #633 is merged is there a way to reproduce/rerun the tests that failed?

@olexandr-konovalov
Copy link
Contributor Author

Now passes - let's observe it a bit more: https://travis-ci.org/gap-infra/gap-docker-pkg-tests-master-staging

@james-d-mitchell
Copy link
Collaborator

Seems like everything passes ok for Semigroups (although everything else seems to fail in your last link!).

@olexandr-konovalov
Copy link
Contributor Author

olexandr-konovalov commented Dec 23, 2019

Yes, because it's a staging test - everything that is known to pass moves to https://travis-ci.org/gap-infra/gap-docker-pkg-tests-master

@james-d-mitchell
Copy link
Collaborator

Só if I make release of Semigroups this will be resolved?

@olexandr-konovalov
Copy link
Contributor Author

olexandr-konovalov commented Dec 28, 2019

It already passes for that Semigroups release which is used in that test, @james-d-mitchell - so you don't have to do anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants