sktime/sktime

[ENH] move test skip config from `tests._config` to estimator tags

Open

#8 515 ouverte le 10 juil. 2025

Voir sur GitHub
 (11 commentaires) (0 réactions) (0 assignés)Python (1 192 forks)batch import
enhancementgood first issuemodule:tests

Métriques du dépôt

Stars
 (7 162 stars)
Métriques de merge PR
 (Merge moyen 26j 10h) (86 PRs mergées en 30 j)

Description

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:

  1. pick one estimator that has an entry in tests._config, in EXCLUDE_ESTIMATORS, EXCLUDED_TESTS, or EXCLUDED_TESTS_BY_TEST
  2. remove from EXCLUDE_ESTIMATORS and instead add the tag "tests:skip_all": True to the estimator (see other examples)
  3. remove from EXCLUDED_TESTS, and instead add the tag "tests:skip_by_name" to the estimator, with value the list of skipped tests.
  4. 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.

Guide contributeur

Direction de recherche
Trouvez un estimateur listé dans `EXCLUDE ESTIMATORS`, `EXCLUDED TESTS` ou `EXCLUDED TESTS BY TEST` de `tests. config`. Supprimez l'entrée du fichier de configuration et ajoutez l'étiquette appropriée (par exemple `tests:skip all` ou `tests:skip by name`) à la définition de la classe Python de l'estimateur. Référez vous aux exemples existants dans la base de code. Assurez vous de ne traiter qu'un seul estimateur par demande de tirage pour simplifier les tests.
Stack technique
python
Domaine
testingdeveloper experience
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPython
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85