Skip to content

MAINT Mark for removal deprecated LinearSVC loss options in 0.23#14373

Merged
qinhanmin2014 merged 4 commits intoscikit-learn:masterfrom
rth:doc-linearsvc
Jul 17, 2019
Merged

MAINT Mark for removal deprecated LinearSVC loss options in 0.23#14373
qinhanmin2014 merged 4 commits intoscikit-learn:masterfrom
rth:doc-linearsvc

Conversation

@rth
Copy link
Copy Markdown
Member

@rth rth commented Jul 15, 2019

The l1 / l2 loss options in LinearSVC were renamed to hinge and squared_hinge in 0.16 and marked for removal in 1.0.

Since it's unclear when 1.0 will happen this moves the removal date to 0.23 0.22 0.23.

@rth rth changed the title MAINT Mark for removal deprecated LinearSVC loss options in 0.23 MAINT Mark for removal deprecated LinearSVC loss options in 0.22 Jul 15, 2019
Copy link
Copy Markdown
Member

@qinhanmin2014 qinhanmin2014 left a comment

Choose a reason for hiding this comment

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

Maybe 0.23 or 0.24? since current stable is 0.21.X.

@rth rth changed the title MAINT Mark for removal deprecated LinearSVC loss options in 0.22 MAINT Mark for removal deprecated LinearSVC loss options in 0.23 Jul 15, 2019
@rth
Copy link
Copy Markdown
Member Author

rth commented Jul 15, 2019

Maybe 0.23 or 0.24? since current stable is 0.21.X.

Yes, of course, thanks. Changed back to 0.23.

Copy link
Copy Markdown
Member

@qinhanmin2014 qinhanmin2014 left a comment

Choose a reason for hiding this comment

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

0.24 and a what's new entry might be less controversial, but I'm OK with current version.

@rth
Copy link
Copy Markdown
Member Author

rth commented Jul 15, 2019

0.24 and a what's new entry might be less controversial, but I'm OK with current version.

Well it was deprecated in 2015, 0.23 will be in 2020-2021. Other libraries typically deprecate something, then remove it after a while. scikit-learn started to indicate the exact deprecation version not that long ago I imagine, this code should have been removed long time ago.

@glemaitre
Copy link
Copy Markdown
Member

Could you document it in what's new (under deprecation maybe), such that we easily spot if for removal in the next release?

@rth
Copy link
Copy Markdown
Member Author

rth commented Jul 16, 2019

Could you document it in what's new (under deprecation maybe)

I can add,

- |API| The "l1" and "l2" options of `loss` parameters in :class:`svm.LinearSVC`,
    :class:`svm.LinearSVR` that were deprecated in v0.16 will be removed in v0.23.
    :pr:`12916` by :user:`Roman Yurchak`_. 

but I don't think it makes much sense, because we don't typically make guarantees as to when a deprecated functionality will be removed (simply because we might very well forget to remove it in v0.23). Also the only thing changing in v0.22 is the content of the deprecation message. It's not an API change and mostly irrelevant to current users.

such that we easily spot if for removal in the next release?

Searching for 0.23 is probably a more reliable way of finding deprecations anyway?

@glemaitre
Copy link
Copy Markdown
Member

Searching for 0.23 is probably a more reliable way of finding deprecations anyway?

Reliable yes but a headache as well :). If everything that should be removed for a version could be documented in a single section, it would ease the search :)

Copy link
Copy Markdown
Member

@qinhanmin2014 qinhanmin2014 left a comment

Choose a reason for hiding this comment

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

I think there's enough consensus.

@qinhanmin2014 qinhanmin2014 merged commit b0fdc4c into scikit-learn:master Jul 17, 2019
@rth rth deleted the doc-linearsvc branch July 17, 2019 14:52
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.

4 participants