Skip to content

Conversation

@trivialfis
Copy link
Member

@trivialfis trivialfis commented Jul 23, 2025

  • Keep the intro simple.
  • Update references to model IO.

@trivialfis trivialfis requested a review from Copilot July 23, 2025 10:12

This comment was marked as outdated.

@trivialfis trivialfis requested review from Copilot and hcho3 July 23, 2025 10:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR splits the support matrix from the introduction documentation to improve documentation structure. The main goal is to keep the intro simple while providing a dedicated section for data input type support information.

  • Moved the comprehensive support matrix to a dedicated data input documentation file
  • Updated documentation references to point to the new location
  • Minor text improvements and corrections in documentation and comments

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
python-package/xgboost/core.py Added polars to supported libraries list and fixed minor typos
doc/python/python_intro.rst Removed support matrix, updated references, and simplified examples
doc/python/index.rst Added reference to new data_input documentation
doc/python/data_input.rst New file containing the complete support matrix moved from intro
Comments suppressed due to low confidence (1)

python-package/xgboost/core.py:956

  • The library name 'polars' should be capitalized as 'Polars' to match the proper naming convention used for other libraries in the list (Pandas, Modin, cuDF).
            as Pandas, Modin, polars, and cuDF), The DMatrix recognizes categorical

@trivialfis trivialfis merged commit 62b5b16 into dmlc:master Jul 26, 2025
79 of 84 checks passed
@trivialfis trivialfis deleted the doc-py-input branch July 26, 2025 11:27
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.

2 participants