fix key error for non rr multiprocess #376
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.
What is the problem / what does the code in this PR do
See issue #372 for the problem.
Can you briefly describe how it works?
When we create a
ParallelSourcePlugin
, there is some error in finding thedtype
for a requested data type. This yields the error as in #372.Usually this is not much of a problem but if we already have raw_records registered, this becomes a mess. I now just iterate over the known plugins to the
ParallelSourcePlugin
to find the right dtype. Given that we have been running like this for a long time illustrates that there are probably few people actually using strax to it's max performance and few will notice.Can you give a minimal working example (or illustrate with a figure)?
I'll upload a
multiprocess problem.ipynb
to the StraxTests on the analysiscode repo.Tests?
Nah don't think such a think is likely to occur again any time soon.