[ENH] reactivate and fix test_multiprocessing_idempotent
#5573
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 PR fixes two untracked issues:
test_multiprocessing_idempotent
would be skipped. A comment said that some issues were related to it, but there was no GitHub issue. This PR removes the skip, checking whether the problem is gone.test_multiprocessing_idempotent
would also fail for forecasters with probabilistic capability, due to the output type not being comparable via the approx array comparison. The modified test now skips forpredict_proba
outputs.The failures would also be thrown when using
check_estimator
, this was noticed by @benHeid forConformalIntervals
.