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

[Docs] Add xgboost version restriction #12050

Merged
merged 1 commit into from
Jul 11, 2022

Conversation

Sunny-Island
Copy link
Contributor

This PR is related to issue #12009

About this PR:

Publish an XGBoost version notice in doc & set up tutorial.

@Sunny-Island Sunny-Island changed the title [Docs]Add xgboost version restriction [Docs] Add xgboost version restriction Jul 11, 2022
Copy link
Member

@zxybazh zxybazh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution! Generally looks OK, would you please make the version change more complete and make sure to use black formatter (version) for python code to pass the linter.

docs/install/from_source.rst Outdated Show resolved Hide resolved
docs/install/from_source.rst Outdated Show resolved Hide resolved
python/gen_requirements.py Outdated Show resolved Hide resolved
python/gen_requirements.py Outdated Show resolved Hide resolved
@Sunny-Island Sunny-Island force-pushed the Sunny-Island/xgboost_doc_fix branch from 633246f to 42a0c60 Compare July 11, 2022 06:55
Copy link
Member

@zxybazh zxybazh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for the document update!

@masahi masahi merged commit fbb7b5d into apache:main Jul 11, 2022
masahi pushed a commit to masahi/tvm that referenced this pull request Jul 15, 2022
Co-authored-by: jiabeizhao <jiabeizhao@tencent.com>
junrushao pushed a commit to yelite/tvm that referenced this pull request Jul 27, 2022
Co-authored-by: jiabeizhao <jiabeizhao@tencent.com>
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
Co-authored-by: jiabeizhao <jiabeizhao@tencent.com>
mikeseven pushed a commit to mikeseven/tvm that referenced this pull request Sep 27, 2023
Co-authored-by: jiabeizhao <jiabeizhao@tencent.com>
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