Skip to content

DEP deprecate loss_function_ attribute in SGDClassifier and SGDOneClassSVM - #27979

Merged
jeremiedbb merged 4 commits into
scikit-learn:mainfrom
lorentzenchr:deprecate_sgd_loss_function
Dec 19, 2023
Merged

jeremiedbb merged 4 commits into
scikit-learn:mainfrom
lorentzenchr:deprecate_sgd_loss_function

Conversation

@lorentzenchr

Copy link
Copy Markdown
Member

Reference Issues/PRs

Helps with #15123 for SGD.

What does this implement/fix? Explain your changes.

Deprecate the attribute loss_function_ that accesses a Cython extension class in SGDClassifier and SGDOneClassSVM.

Any other comments?

Let's try to include it in 1.4.

@github-actions

github-actions Bot commented Dec 18, 2023 •

Copy link
Copy Markdown

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: dd9e0cc. Link to the linter CI: here

@lorentzenchr lorentzenchr added this to the 1.4 milestone Dec 18, 2023
@lorentzenchr

lorentzenchr commented Dec 18, 2023 •

Copy link
Copy Markdown
Member Author

@glemaitre @lesteve @jeremiedbb @jjerphan As we won't rush 1.4. anyway, could we get this deprecation in? 🙏

@jjerphan jjerphan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you, @lorentzenchr.

@jeremiedbb jeremiedbb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. I just pushed a few changes for consistency in version numbers

@jeremiedbb
jeremiedbb enabled auto-merge (squash) December 19, 2023 13:49
@jeremiedbb
jeremiedbb merged commit 11b45b3 into scikit-learn:main Dec 19, 2023
@lorentzenchr

Copy link
Copy Markdown
Member Author

@jjerphan @jeremiedbb Thanks for the quick review!
@jeremiedbb Thanks for proactively taking care of nitpicks.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants