Skip to content

[MRG+1] removing deprecated files in metrics#5456

Merged
amueller merged 2 commits intoscikit-learn:masterfrom
aabadie:remove_deprecation_metrics
Oct 21, 2015
Merged

[MRG+1] removing deprecated files in metrics#5456
amueller merged 2 commits intoscikit-learn:masterfrom
aabadie:remove_deprecation_metrics

Conversation

@aabadie
Copy link
Copy Markdown
Contributor

@aabadie aabadie commented Oct 19, 2015

Partly fixed metrics deprecated code: removed metrics.py file
#5434

@aabadie
Copy link
Copy Markdown
Contributor Author

aabadie commented Oct 20, 2015

@ogrisel, this PR fixes the deprecation message in metrics.py

@amueller
Copy link
Copy Markdown
Member

This was correct. It is part of #3442. If you git checkout 0.16.X you can see that it was already deprecated there. It's unfortunate that there was no whatsnew message.

@aabadie
Copy link
Copy Markdown
Contributor Author

aabadie commented Oct 21, 2015

@mueller, my understanding is that the message should be updated in 0.17.X branch (the corresponding commit will then be cherry-picked from master) and, still in master, remove the metrics.py file.

Do you confirm ?

Alexandre Abadie added 2 commits October 21, 2015 13:13
confirm deprecation starting from 0.16 and removal after 0.18
@amueller
Copy link
Copy Markdown
Member

It should be removed in master. I don't think anything needs to be updated in 0.17.X.

@raghavrv
Copy link
Copy Markdown
Member

@GaelVaroquaux @ogrisel This can also be merged!

@GaelVaroquaux GaelVaroquaux changed the title [MRG] removing deprecated files in metrics [MRG+1] removing deprecated files in metrics Oct 21, 2015
@GaelVaroquaux
Copy link
Copy Markdown
Member

LGTM. +1 for merge from my side.

amueller added a commit that referenced this pull request Oct 21, 2015
[MRG+1] removing deprecated files in metrics
@amueller amueller merged commit ebf8d3a into scikit-learn:master Oct 21, 2015
@amueller
Copy link
Copy Markdown
Member

maybe should have squashed. well whatever. Thanks @aabadie

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