-
Notifications
You must be signed in to change notification settings - Fork 171
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
Core dump with VietorisRipsPersistence and joblib #59
Comments
Possibly related to joblib/joblib#849 |
Hi, Thanks for the feedback. Could you provide your joblib version? |
joblib 0.14.0 |
@arksch was this ever resolved? Is it still happening with the latest version of |
I'm getting this issue as well. |
@jlevy44 thanks for the report! Could you please provide us with your setup and with a minimum working example so we can try to reproduce this issue? To get the setup, please run the following snippet and paste the output:
|
Sounds good.
Instead of posting a working example, I can send you my solution, which I was able to get to run! I ended up hacking your script and replacing the joblib backend with dask:
This worked as intended. :) |
@jlevy44 thanks for reporting on your setup and it's interesting that you found a solution with dask! However, it is still important to have a minimum working example (if you are able to provide it), to be able to better pinpoint the exact origin of the problem. Since we use |
Additionally, I'm wondering if you were passing a collection with a single sample even before finding the solution with dask. |
No, I wasn't, if I'm understanding you correctly. Sometimes it (joblib backend) would work (especially if I reduced the number of point clouds to ~30), other times it would fail (especially close to 100 clouds and higher max_edge_length). I am unable to provide our working example right now, apologies! |
@jlevy44 thank you, I understand that you cannot provide this example. Could you perhaps give me information on the shape of the point clouds you use? |
@jlevy44 aside from my above question about shapes and number of point clouds used (so I can try my best to reproduce), I'm wondering if you can try the following since you don't seem scared by the idea of changing the source code: in the call to |
I cannot reproduce it with the newest version. Solved from my side. Thanks! Linux-4.15.0-117-generic-x86_64-with-Ubuntu-18.04-bionic |
Closing as this is likely no longer an issue with the |
Description
Core dump when calling fit_transform on VietorisRipsPersistence with n_jobs=None or 1,
TerminatedWorkerError when n_jobs=2.
Steps/Code to Reproduce
Expected Results
No error is thrown.
Actual Results
Illegal instruction (core dumped)
Unreportable Reason: Cannot determine path of python module joblib.externals.loky.backend.popen_loky_posix"
Or if n_jobs=1:
TerminatedWorkerError: A worker process managed by the executor was unexpectedly terminated. This could be caused by a segmentation fault while calling the function or by an excessive memory usage causing the Operating System to kill the worker. The exit codes of the workers are {SIGILL(-4)}
Versions
Linux-4.15.0-65-generic-x86_64-with-Ubuntu-18.04-bionic
Python 3.6.8 (default, Oct 7 2019, 12:59:55)
[GCC 8.3.0]
NumPy 1.17.2
SciPy 1.3.1
Scikit-Learn 0.21.3
giotto-Learn 0.1.1
The text was updated successfully, but these errors were encountered: