Skip to content
This repository has been archived by the owner on Feb 20, 2024. It is now read-only.

import multiprocessing as mp #12

Closed
wants to merge 1 commit into from

Conversation

WhiteSymmetry
Copy link

Windows fails when you do not type this command. When I write it this way, it works without errors.

Pull Request Template

This template should be used for making any pull requests.

Goal

Here, describe the goal of the pull request, i.e. what does it seek to address?
Does this pull request fix an existing bug, or create a new feature?
Why are the changes proposed by this pull request necessary?

Specifics

Here, describe the specific changes to the repository suggested by this pull request.
Which modules/methods/etc. were added or edited, and why?
Have any new dependencies been introduced, and why?

Verification

Here, describe steps taken to verify that the pull request is up to the standards of the repository.
Were the suggested changes fully documented and tested, and do they match the existing code style?
How exactly was this verification completed?

Windows fails when you do not type this command. When I write it this way, it works without errors.
@mrbuche
Copy link
Collaborator

mrbuche commented Apr 11, 2022

@WhiteSymmetry

Thanks for bringing this up. multiprocessing is disabled for Windows/MacOS currently since it doesn't always work for everyone. The code in the repo now should avoid the issue you experienced (the use of multiprocessing as mp is now within an OS check).

@mrbuche mrbuche closed this Apr 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants