sktime/sktime

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

Open

#8515 aperta il 10 lug 2025

Vedi su GitHub
 (11 commenti) (0 reazioni) (0 assegnatari)Python (1192 fork)batch import
enhancementgood first issuemodule:tests

Metriche repository

Star
 (7162 star)
Metriche merge PR
 (Merge medio 26g 10h) (86 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Trova un estimatore elencato in `EXCLUDE ESTIMATORS`, `EXCLUDED TESTS` o `EXCLUDED TESTS BY TEST` di `tests. config`. Rimuovi la voce dal file di configurazione e aggiungi l'etichetta appropriata (ad esempio `tests:skip all` o `tests:skip by name`) alla definizione della classe Python dell'estimatore. Fai riferimento agli esempi esistenti nella codebase. Assicurati di gestire un solo estimatore per pull request per semplificare i test.
Tech stack
python
Dominio
testingdeveloper experience
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85