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.

Sign up for GitHub

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

Jump to bottom

Require fork for multiprocessing #100

Merged
merged 1 commit into from
Apr 7, 2023
Merged

Require fork for multiprocessing #100

merged 1 commit into from
Apr 7, 2023

Conversation

arahlin
Copy link
Member

@arahlin arahlin commented Apr 7, 2023

This avoids errors with pickling the target module when running in a subprocess.

Fixes #86.

@arahlin arahlin requested a review from nwhitehorn April 7, 2023 14:20
@arahlin arahlin self-assigned this Apr 7, 2023
@arahlin arahlin added the bug label Apr 7, 2023
@arahlin
Require fork for multiprocessing
09dd137 
This avoids errors with pickling the target module when running in a subprocess.

Fixes #86.
nwhitehorn
nwhitehorn approved these changes Apr 7, 2023
View reviewed changes
Copy link
Member

@nwhitehorn nwhitehorn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for tracking this down! Looks good.

@arahlin arahlin merged commit 8e4817d into master Apr 7, 2023
@arahlin arahlin deleted the multiproc_fork branch April 7, 2023 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nwhitehorn nwhitehorn nwhitehorn approved these changes

Assignees

@arahlin arahlin

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cannot run pipeline modules in a subprocess with python versions > 3.7
2 participants
@arahlin @nwhitehorn