Skip to content

DOC Rework LinearRegression documentation#7218

Merged
rapids-bot[bot] merged 18 commits intorapidsai:mainfrom
virchan:issues/1447/linear_regression
Dec 1, 2025
Merged

DOC Rework LinearRegression documentation#7218
rapids-bot[bot] merged 18 commits intorapidsai:mainfrom
virchan:issues/1447/linear_regression

Conversation

@virchan
Copy link
Copy Markdown
Contributor

@virchan virchan commented Sep 14, 2025

Reference Issues/PRs

Towards #1447

What does this implement/fix? Explain your changes.

Rework the Linear Regression documentation.

Additionally, adds cudf, numpy, and sklearn to the Sphinx configuration file to enable cross-library references in the future.

Any other comments?

N/A

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Sep 14, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the Cython / Python Cython or Python issue label Sep 14, 2025
@virchan virchan marked this pull request as ready for review September 14, 2025 07:31
@virchan virchan requested a review from a team as a code owner September 14, 2025 07:31
@virchan virchan requested a review from divyegala September 14, 2025 07:31
@lowener lowener added doc Documentation non-breaking Non-breaking change labels Sep 16, 2025
@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Sep 16, 2025

/ok to test 50c19fe

@virchan
Copy link
Copy Markdown
Contributor Author

virchan commented Sep 17, 2025

Thank you for running the CI!

It flagged a whitespace issue, which was caused by me forgetting to run pre-commit 🤦. I’ve fixed it, and hopefully the CI will be all green this time 🤞.

@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Sep 17, 2025

/ok to test 4636bbc

@virchan
Copy link
Copy Markdown
Contributor Author

virchan commented Sep 22, 2025

If it's not too much trouble, could we re-run the CI for this PR? I believe the Pytest failure related to XGBoost has already been addressed in #7230, but please correct me if I’m mistaken.

@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Sep 23, 2025

/ok to test 7fe082f

@virchan virchan changed the base branch from branch-25.10 to branch-25.12 October 2, 2025 16:31
@virchan
Copy link
Copy Markdown
Contributor Author

virchan commented Oct 2, 2025

If it's not too much trouble, could we re-run the CI for this PR? I believe the failure in 7fe082f has been resolved in #7257, but please correct me if I’m mistaken. I have also updated the branch from 25.10 to 25.12.

@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Oct 3, 2025

/ok to test 8f48cd7

@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Nov 5, 2025

/ok to test 20b3b53

@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Nov 14, 2025

/ok to test 7ca93fa

@virchan
Copy link
Copy Markdown
Contributor Author

virchan commented Nov 18, 2025

If it's not too much trouble, could we re-run the CI for this PR?

@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Nov 18, 2025

@divyegala Please review prior to running tests.

Comment thread python/cuml/cuml/linear_model/linear_regression.pyx Outdated
@divyegala
Copy link
Copy Markdown
Member

/ok to test c7c7d79

@divyegala
Copy link
Copy Markdown
Member

/merge

@rapids-bot rapids-bot Bot merged commit 337324e into rapidsai:main Dec 1, 2025
275 of 286 checks passed
@virchan
Copy link
Copy Markdown
Contributor Author

virchan commented Dec 1, 2025

Thank you everyone for your time! 🫡

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cython / Python Cython or Python issue doc Documentation non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOC] A few classes are missing usage examples. Many others could be updated.

6 participants