Skip to content

MAINT: remove deprecated multi_class="multinomial" from LogisticRegression usage#1147

Merged
rasbt merged 11 commits into
rasbt:masterfrom
sachinn854:remove-multiclass-param
Jan 8, 2026
Merged

MAINT: remove deprecated multi_class="multinomial" from LogisticRegression usage#1147
rasbt merged 11 commits into
rasbt:masterfrom
sachinn854:remove-multiclass-param

Conversation

@sachinn854
Copy link
Copy Markdown
Contributor

This PR removes explicit multi_class="multinomial" arguments from
LogisticRegression usage in examples / notebooks / tests.

In sklearn ≥ 1.5 the multi_class parameter is deprecated and will be
removed in 1.7. The default behavior is now always multinomial, so
removing this avoids FutureWarning while keeping identical behavior.

✓ verified on sklearn ≥ 1.6
✓ warnings disappear
✓ outputs unchanged

Note: usages with multi_class="ovr" were intentionally left unchanged
to avoid altering behavior.

Closes #1126

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@sachinn854 sachinn854 force-pushed the remove-multiclass-param branch from 8cfa07f to 7b7ef97 Compare December 28, 2025 08:19
Comment thread mlxtend/classifier/perceptron.py Outdated
Comment thread mlxtend/feature_selection/tests/test_exhaustive_feature_selector.py Outdated
Comment thread mlxtend/feature_selection/tests/test_sequential_feature_selector.py Outdated
Copilot AI review requested due to automatic review settings January 8, 2026 15:10
Copy link
Copy Markdown

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@rasbt rasbt merged commit 706320f into rasbt:master Jan 8, 2026
3 checks passed
@sachinn854
Copy link
Copy Markdown
Contributor Author

sachinn854 commented Jan 8, 2026

Thanks for the review and merge.

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.

LogisticRegression: 'multi_class' param

3 participants