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

[LogisticRegressionMG] Support standardization #5656

Closed

Conversation

lijinf2
Copy link
Contributor

@lijinf2 lijinf2 commented Nov 15, 2023

This PR adds an optional parameter to support standardizing the training dataset before fitting. It helps match the behavior of Apache Spark LogisticRegression.

The PR depends on #5655

@lijinf2 lijinf2 requested review from a team as code owners November 15, 2023 06:23
Copy link

copy-pr-bot bot commented Nov 15, 2023

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 Cython / Python Cython or Python issue CUDA/C++ labels Nov 15, 2023
@lijinf2 lijinf2 added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Nov 15, 2023
@lijinf2 lijinf2 force-pushed the fea_lrmg_standardization branch 2 times, most recently from 76432b6 to f6eb61a Compare November 16, 2023 22:44
@lijinf2 lijinf2 force-pushed the fea_lrmg_standardization branch 6 times, most recently from 457d7aa to a4a8af9 Compare November 29, 2023 03:07
@lijinf2 lijinf2 changed the base branch from branch-23.12 to branch-24.02 December 1, 2023 20:12
@lijinf2 lijinf2 force-pushed the fea_lrmg_standardization branch from a4a8af9 to 04e3caf Compare December 1, 2023 23:51
@lijinf2 lijinf2 force-pushed the fea_lrmg_standardization branch from 04e3caf to 67da440 Compare December 1, 2023 23:52
@lijinf2 lijinf2 closed this Mar 18, 2024
@lijinf2 lijinf2 deleted the fea_lrmg_standardization branch June 26, 2024 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CUDA/C++ Cython / Python Cython or Python issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants