Test skip configs should be moved from tests._config to estimator tags.
This is a good first issue with a recipe applicable to many cases:
- pick one estimator that has an entry in
tests._config, in EXCLUDE_ESTIMATORS, EXCLUDED_TESTS, or EXCLUDED_TESTS_BY_TEST
- remove from
EXCLUDE_ESTIMATORS and instead add the tag "tests:skip_all": True to the estimator (see other examples)
- remove from
EXCLUDED_TESTS, and instead add the tag "tests:skip_by_name" to the estimator, with value the list of skipped tests.
- if the estimator appears in
EXCLUDED_TESTS_BY_TEST as well, try to fix get_test_params or the docstring (do not transfer this otherwise).
Please only handle a single estimator at a time, since the tests will trigger in the pull request.
Test skip configs should be moved from
tests._configto estimator tags.This is a good first issue with a recipe applicable to many cases:
tests._config, inEXCLUDE_ESTIMATORS,EXCLUDED_TESTS, orEXCLUDED_TESTS_BY_TESTEXCLUDE_ESTIMATORSand instead add the tag"tests:skip_all": Trueto the estimator (see other examples)EXCLUDED_TESTS, and instead add the tag"tests:skip_by_name"to the estimator, with value the list of skipped tests.EXCLUDED_TESTS_BY_TESTas well, try to fixget_test_paramsor the docstring (do not transfer this otherwise).Please only handle a single estimator at a time, since the tests will trigger in the pull request.