Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Issue 34 #43

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

Conversation

supercoder-dev
Copy link

To solve the multiprocessing issue, we need to wrap the call to self._build_roots(X, y, weights) inside an if __name__ == '__main__': block. This ensures that the multiprocessing module works correctly when n_jobs is greater than one.
To solve the problem, we need to modify the fork_call function to use multiprocessing.get_context('fork') for creating the process. Additionally, we need to ensure that the process creation is wrapped in a if __name__ == '__main__': block to avoid the RuntimeError.

@Refefer
Copy link
Owner

Refefer commented Jun 19, 2024

I'll try it out today; on first blush the changes seem reasonable.

It's also clearly time to drop python2 support.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants