sktime/sktime
Auf GitHub ansehen[ENH] move test skip config from `tests._config` to estimator tags
Open
#8.515 geöffnet am 10. Juli 2025
enhancementgood first issuemodule:tests
Repository-Metriken
- Stars
- (7.162 Stars)
- PR-Merge-Metriken
- (Durchschn. Merge 26T 10h) (86 gemergte PRs in 30 T)
Beschreibung
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, inEXCLUDE_ESTIMATORS,EXCLUDED_TESTS, orEXCLUDED_TESTS_BY_TEST - remove from
EXCLUDE_ESTIMATORSand instead add the tag"tests:skip_all": Trueto 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_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.