Measuring runtimes #1031
Measuring runtimes #1031
Conversation
tests/test_extensions/test_sklearn_extension/test_sklearn_extension.py
Outdated
Show resolved
Hide resolved
mfeurer
left a comment
mfeurer
left a comment
There was a problem hiding this comment.
Looks good, except for the failure that you mentioned offline
Should pass with the changes made. |
mfeurer
left a comment
mfeurer
left a comment
There was a problem hiding this comment.
I think you can also remove the flow in this PR as it is not needed and removing it is helpful in reaching this PRs goal.
|
Added a |
| res = format_prediction(regression, *ignored_input) | ||
| self.assertListEqual(res, [0] * 5) | ||
|
|
||
| @pytest.mark.flaky() # appears to fail stochastically on test server |
There was a problem hiding this comment.
Are these errors only with loky? If so, we could use multiprocessing as a default here as we don't want to test for a specific backend in this unit test?
* [skip ci] addressing openml#248 * Unit test to test existence of refit time * Refactoring unit test * Fixing unit test failures * Unit test fixing + removing redundant parameter * Debugging stochastic failure of test_joblib_backends unit test * Unit test fix with decorators * Flaky for failing unit test * Adding flaky reruns for unit tests * Fixing setup big * pytest rerun debug * Fixing coverage failure * Debugging coverage failure * Debugging coverage failure * Adding __init__ files in test/ for pytest-cov * Debugging coverage failure * Debugging lean unit test * Debugging loky failure in unit tests * Clean up of debugging stuff
Reference Issue
Addresses #248.