Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Relax test for decision stump in distributed environment. #6919

Merged
merged 2 commits into from
Apr 30, 2021

Conversation

trivialfis
Copy link
Member

This relaxes the test when the max_bin is set to 2 and trees are limited to stumps. In this case, the floating-point error can overweight the learning result.

https://xgboost-ci.net/blue/organizations/jenkins/xgboost/detail/PR-6901/6/pipeline

@trivialfis trivialfis mentioned this pull request Apr 29, 2021
6 tasks
@trivialfis trivialfis merged commit a1d23f6 into dmlc:master Apr 30, 2021
@trivialfis trivialfis deleted the relax-stump branch April 30, 2021 01:04
trivialfis added a commit to trivialfis/xgboost that referenced this pull request May 3, 2021
trivialfis added a commit that referenced this pull request May 3, 2021
* Relax shotgun test. (#6900)

It's non-deterministic algorithm, the test is flaky.

* Disable pylint error. (#6911)

* [CI] Skip external memory gtest on osx. (#6901)

* [CI] Fix custom metric test with empty dataset. (#6917)

* Reduce Travis environment setup time. (#6912)

* Remove unused r from travis.
* Don't update homebrew.
* Don't install indirect/unused dependencies like libgit2, wget, openssl.
* Move graphviz installation to conda.

* Relax shotgun test. (#6918)

* Relax test for decision stump in distributed environment. (#6919)

* Backport cupy fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants