Propagate the parallelize option in open_run() #338
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This comes up when using
open_run(data="all")
, which will make another callinternally to load proc data with
open_run(data="proc")
. Since theparallelize option wasn't passed, the call to open proc data would use the
default of
parallelize=True
.Note: I refactored
test_open_run()
to re-use the code for creating a mock root directory with raw and proc directories etc, but I'm not sure I like how messy it is to return so many paths from a fixture 馃し I could also copy and paste the setup lines intotest_open_run_daemonized()
.