alteryx / alteryx/evalml

Revert changes to testing with new XGBoost release

オープン
#3,331 コメント 0 件 リアクション 0 件 担当者 1 名 @fjlanasa が担当を希望しています GitHub で見る
good first issue tech debt testing
主要言語
Python
スター
850
フォーク
96
PR マージ指標
30日以内にマージされた PR はありません

説明

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](https://github.com/dmlc/xgboost/pull/7595) 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.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。