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

[Fix] multiprocessing bug with pickle #69

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mbml84
Copy link

@mbml84 mbml84 commented Jan 25, 2023

No description provided.

Gabri95 and others added 2 commits January 25, 2023 08:04
Signed-off-by: Maxime Boulanger <[email protected]>
Signed-off-by: Maxime Boulanger <[email protected]>
@mbml84 mbml84 changed the title Bugfix/multiprocessing signoff Bugfix/multiprocessing Jan 25, 2023
@mbml84
Copy link
Author

mbml84 commented Jan 25, 2023

Fix a pickle bug when using multiprocessing

@mbml84 mbml84 changed the title Bugfix/multiprocessing [Fix] multiprocessing bug with pickle Jan 25, 2023
@Gabri95
Copy link
Collaborator

Gabri95 commented Jan 30, 2023

Hi @mbml84

Thanks for the pull request!

I am actually having a similar issue in the new escnn library.

Your code should fix the issue here but it seems to involve some large changes in the backend, so I think I need to take some extra care to integrate them in the code base.

Because I am already working on this in the new version of the library and I am trying to discontinue e2cnn, I would point you to escnn.
Your pull request seems useful and I'd like it if you opened a similar one there (although I understand that might be some work for you, sorry).

If, for compatibility issues, more people need this feature in e2cnn, I will try to fix the issue here too.

Thanks again,
Gabriele

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

Successfully merging this pull request may close these issues.

2 participants