Skip to content

[MRG] remove deprecated stuff from 0.17 #4370

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

Merged
merged 2 commits into from
Mar 18, 2015

Conversation

amueller
Copy link
Member

@amueller amueller commented Mar 9, 2015

This PR removes most of the deprecated things for 0.17.
Two items are left afterwards:

  • sequence of sequence multi-label format. I wasn't so sure what to touch here, and maybe @arjoly or @jnothman want to go ahead with it.
  • possibly refactoring cv to not accept boolean masks.

I'm not sure if we want cross_val_score and GridSearchCV to support iterators that support boolean masks. I think the intention was to remove this support completely, but I'm not sure. For now, I only removed the option to return boolean masks from the cross-validation iterators.

@amueller amueller force-pushed the 0.17_deprecated branch 4 times, most recently from ca6ab01 to ed1f093 Compare March 10, 2015 18:33
@amueller
Copy link
Member Author

I tried to remove the multi-label support from the multi-label generator, but even that was non-trivial. @arjoly your help would be much appreciated, maybe in a different pr?

@amueller amueller force-pushed the 0.17_deprecated branch 3 times, most recently from 012efe7 to fcfe3c5 Compare March 10, 2015 18:56
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 95.08% when pulling fcfe3c5 on amueller:0.17_deprecated into 5f2faac on scikit-learn:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 95.08% when pulling fcfe3c5 on amueller:0.17_deprecated into 5f2faac on scikit-learn:master.

@raghavrv
Copy link
Member

Haven't we disabled coveralls bot already??

@coveralls shoo go away! ;)

@ogrisel
Copy link
Member

ogrisel commented Mar 18, 2015

Great cleanup, merging. Indeed the remaining non-trivial stuff should better be addressed in separate PRs.

ogrisel added a commit that referenced this pull request Mar 18, 2015
[MRG] remove deprecated stuff from 0.17
@ogrisel ogrisel merged commit c4df19d into scikit-learn:master Mar 18, 2015
@ogrisel
Copy link
Member

ogrisel commented Mar 18, 2015

BTW @amueller this kind of commits could be prefixed with a MAINT header.

@ogrisel
Copy link
Member

ogrisel commented Mar 18, 2015

-23k loc! 🍻

@amueller amueller deleted the 0.17_deprecated branch March 18, 2015 20:09
@amueller
Copy link
Member Author

yep, should have been MAINT.

@amueller
Copy link
Member Author

The locs are mostly generated C for hmm ^^

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