-
Notifications
You must be signed in to change notification settings - Fork 664
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
Test failing locally: test_parallel_calculation #2606
Comments
Hi @orbeckst |
@shfrz I also suspect that my local environment is faulty. Did you test it with the same Python and OS as me?
|
So I can't reproduce the error using py 3.8.1 (conda hasn't released a main package of 3.8.2 yet). That being said unless someone else tests things with 3.8.2 (particularly on Catalina), it's not possible to tell if this is a reproducible issue or not. Pinging @RMeli here possibly, as known MacOS user 😉 |
I can confirm this issue on macOS 10.15.3 with Python 3.8.2 and on the developer branch (a2951ee). All dependencies apart from |
To add on this, this issue is not related to |
Yes, Travis CI didn't catch this issue. I don't know how it works exactly, but it may not be testing with this particular case of OS - Python combination. |
It looks like our Travis test matrix only covers py3.6 for osx (linux covers the whole range), which is probably why this didn't get found earlier. |
Thanks for the report- I can reproduce on macOS Catalina and Python 3.8.2 - @IAlibay @abhiShandy #2615 effectively fixes it for me, thanks for this! |
* Fixes #2606 Pytest and/or multiprocessing fails to pickle local functions. Found a workaround here: https://bugs.python.org/issue33884 (Basically, moving the local function to the top level.) * the function which is being evaluated in parallel is global now * make parallel test xfail for windows (because even this workaround does not help under windows)
Expected behavior
No failed test
Actual behavior
platform darwin -- Python 3.8.2, pytest-5.3.5, py-1.8.1, pluggy-0.13.1
rootdir: /Users/abhishandy/Code/mdanalysis/fork/testsuite, inifile: setup.cfg
plugins: forked-1.1.2, xdist-1.31.0, hypothesis-5.6.0
Code to reproduce the behavior
pytest --disable-pytest-warnings --numprocesses 4
Current version of MDAnalysis
The text was updated successfully, but these errors were encountered: