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

Support running tests in parallel #92

Closed
bwohlberg opened this issue Nov 14, 2021 · 2 comments
Closed

Support running tests in parallel #92

bwohlberg opened this issue Nov 14, 2021 · 2 comments
Labels
priority: low Low priority tests Pertaining to SCICO tests

Comments

@bwohlberg
Copy link
Collaborator

Since our test suite takes quite a while to run, it would be convenient to be able to run the tests in parallel using either pytest-xdist or pytest-parallel. Unfortunately neither of these pytest plugins works at the moment

@bwohlberg bwohlberg added tests Pertaining to SCICO tests priority: low Low priority labels Nov 14, 2021
@FernandoDavis
Copy link
Contributor

The following are the only tests that do not work with x-dist:

  • scico/test/test_admm.py
    Fails due to mismatch of results. Not sure if directly related to x-dist or if it's a mistake in my current build.
  • scico/test/test_functional.py
    Different tests were collected between workers. This is likely due to concurrency issues in data between test classes/methods.

After I finish taking care of other issues I will proceed and fix these two modules so that they are compatible. Could be a big or small change though, will update this thread on findings.

@bwohlberg
Copy link
Collaborator Author

Closing for now since this is largely addressed in #390.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: low Low priority tests Pertaining to SCICO tests
Projects
None yet
Development

No branches or pull requests

2 participants