alteryx/evalml

Revert changes to testing with new XGBoost release

Aperta

#3331 aperta il 15 feb 2022

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (1 assegnatario)Python (93 fork)auto 404
good first issuetech debttesting

Metriche repository

Star
 (852 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

In #3324 , we had to mark some tests as expected to fail since XGBoost was throwing a FutureWarning. The warning has been addressed in XGBoost, so we're just waiting for the PR merged to be released. This issue is discussed in the #3275 issue.

evalml/tests/component_tests/test_xgboost_classifier.py needs to have the @pytest.mark.xfail removed from the test_xgboost_catch_warnings_label_encoder test when XGBoost releases. EvalML will upgrade automatically when XGBoost releases, which should actually cause these tests to pass (and thus fail because of the xfail mark). Once the xfail is removed, this story is complete.

Guida contributor

Direzione di ricerca
Controlla se la nuova release di XGBoost è disponibile, quindi rimuovi il decoratore @pytest.mark.xfail dal test test xgboost catch warnings label encoder e verifica che il test passi.
Tech stack
python
Dominio
testing qa
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85