Skip to content

[MRG] DOC cleaning up what's new for 0.19 #9252

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 26 commits into from
Jul 13, 2017

Conversation

jnothman
Copy link
Member

@jnothman jnothman commented Jun 30, 2017

subsections, reclassifications, etc

Marked WIP only because of optimistic inclusion of multiple metrics, which is not yet merged.

:issue:`9105` by `Hanmin Qin <https://github.com/qinhanmin2014>`_.
- :class:`neighbors.LSHForest` has been deprecated and will be
removed in 0.21 due to poor performance.
:issue:`8996` by `Andreas Müller`_.
Copy link
Member

Choose a reason for hiding this comment

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

This item does not seem to belong to "Linear, kernelized and related models".

Copy link
Member

Choose a reason for hiding this comment

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

issue by me, but not the fix, so probably should be a different name?

would fail with a unmeaningful error by default.
:issue:`6573` by :user:`Quazi Marufur Rahman <qmaruf>` and
`Manoj Kumar`_.
Linear, kernelized and related models
Copy link
Member

Choose a reason for hiding this comment

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

Maybe we should have one section name "Neighborhood models" and another "Semi-supervised learning models" instead?

@jnothman
Copy link
Member Author

jnothman commented Jul 4, 2017 via email

@jnothman
Copy link
Member Author

jnothman commented Jul 4, 2017 via email

@jnothman
Copy link
Member Author

jnothman commented Jul 4, 2017

Unless we want to completely group by subpackage, which might make sense but seems unnecessarily fine-grained for this purpose, I've just added "other predictors".

I had included neighbors and semi_supervised under "linear, kernelised et al" because of how they use affinities. But so does clustering. And so do kernels. And most of our decompositions are linear. So we're splitting hairs in any case. Would you rather I group by subpackage?

@amueller
Copy link
Member

amueller commented Jul 5, 2017

can you merge it with the 0.18.2 whatsnew?

- The ``n_topics`` parameter of :class:`decomposition.LatentDirichletAllocation`
has been renamed to ``n_components`` and will be removed in version 0.21.
:issue:`8922` by :user:`Attractadore`
Metrics
Copy link
Member

Choose a reason for hiding this comment

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

This section is empty, should it be removed?

@lesteve
Copy link
Member

lesteve commented Jul 6, 2017

I push a couple of changes but apart from the empty "Metrics" section, LGTM.

Copy link
Member

@amueller amueller left a comment

Choose a reason for hiding this comment

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

Are you sure we should release? That really seems like to little to warrant a new version ;)

@@ -1391,6 +1389,18 @@ Recently deprecated
===================


To be removed in 0.21
Copy link
Member

Choose a reason for hiding this comment

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

nitpickiest nitpick: shouldn't that go below removed in 0.20? though no strong opinion.

Copy link
Member Author

Choose a reason for hiding this comment

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

Mostly the point of listing them is so that autodoc generates. Order shmorder.

We are excited to release a number of great new features including
:class:`neighbors.LocalOutlierFactor` for anomaly detection,
:class:`preprocessing.QuantileTransformer` for robust feature
transformation, and :class:`multioutput.ClassifierChain` to simply
Copy link
Member

Choose a reason for hiding this comment

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

Maybe say that ClassifierChain is a meta-estimator?

account for dependencies between classes in multilabel problems. We
have some new algorithms in existing estimators, such as
multiplicative update in :class:`decomposition.NMF` and multinomial
:class:`linear_model.LogisticRegression` with L1 loss.
Copy link
Member

Choose a reason for hiding this comment

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

via the saga solver?

you can predict faster: if you're sure you know what you're doing, you can turn
off validating that the input is finite using :func:`config_context`.

Cross validation is now able to return the results from multiple metric
Copy link
Member

Choose a reason for hiding this comment

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

that's not merged yet, right?

Copy link
Member Author

Choose a reason for hiding this comment

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

no, it's not. Nor are the TSNE fixes. Being optimisitc, and getting the writing out of the way.

:class:`manifold.TSNE` implementation have been fixed, particularly in the
default Barnes-Hut approximation. :class:`semi_supervised.LabelSpreading` and
:class:`semi_supervised.LabelPropagation` have had substantial fixes.
Propagation was previously broken. Spreading should now correctly respect its
Copy link
Member

Choose a reason for hiding this comment

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

LabelPropagation?

- Fixed a bug in :class:`linear_model.RandomizedLasso`,
:class:`linear_model.Lars`, :class:`linear_model.LassoLars`,
:class:`linear_model.LarsCV` and :class:`linear_model.LassoLarsCV`,
where the parameter ``precompute`` were not used consistently across
Copy link
Member

Choose a reason for hiding this comment

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

wast not?

where a float being compared to ``0.0`` using ``==`` caused a divide by zero
error. issue:`7970` by :user:`He Chen <chenhe95>`.
- Fix a bug where :func:`linear_model.LassoLars.fit` sometimes
left `coef_` as a list, rather than an ndarray.
Copy link
Member

Choose a reason for hiding this comment

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

double backticks?

array X and initial centroids, where X's means were unnecessarily being
subtracted from the centroids. :issue:`7872` by :user:`Josh Karnofsky <jkarno>`.
- Fix :func:`linear_model.BayesianRidge.fit` to return
ridge parameter `alpha_` and `lambda_` consistent with calculated
Copy link
Member

Choose a reason for hiding this comment

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

double backtricks everywhere?

- Fix output shape and bugs with n_jobs > 1 in
:class:`decomposition.SparseCoder` transform and
:func:`decomposition.sparse_encode`
for one-dimensional data and one component.
This also impacts the output shape of :class:`decomposition.DictionaryLearning`.
:issue:`8086` by `Andreas Müller`_.

- Fixed the implementation of `explained_variance_`
Copy link
Member

Choose a reason for hiding this comment

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

double backticks?

:issue:`9105` by `Hanmin Qin <https://github.com/qinhanmin2014>`_.
- :class:`neighbors.LSHForest` has been deprecated and will be
removed in 0.21 due to poor performance.
:issue:`8996` by `Andreas Müller`_.
Copy link
Member

Choose a reason for hiding this comment

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

issue by me, but not the fix, so probably should be a different name?

@jnothman
Copy link
Member Author

jnothman commented Jul 7, 2017

Are you sure we should release? That really seems like to little to warrant a new version ;)

Maybe we've missed something? Make sure you go back and check the logs before giving this LGTM. :P

@amueller
Copy link
Member

why are there conflicts in the tsne test?

@jnothman
Copy link
Member Author

Weird. Must have been a merge error on my part. I'll fix it up shortly

@jnothman jnothman changed the title [WIP] DOC cleaning up what's new for 0.19 [MRG] DOC cleaning up what's new for 0.19 Jul 12, 2017
@jnothman
Copy link
Member Author

Marking MRG

And other minor things.

This took lots of effort which I would have not committed where I not home sick...
@jnothman jnothman merged commit ba72248 into scikit-learn:master Jul 13, 2017
@jnothman jnothman deleted the whatsnew-0.19 branch July 13, 2017 05:54
@jnothman
Copy link
Member Author

Okay. Let's do a release, then :)

massich pushed a commit to massich/scikit-learn that referenced this pull request Jul 13, 2017
* DOC cleaning up what's new for 0.19

* More cleaning up

* More cleaning up

* Deprecations

* Clean up merge

* Update

* TODOs to prose and minor changes

* Changed models and minor fixes

* sort

* Merge in 0.18.2 docs

* Missing entry from 0.18 logs

* Optimistically add some features to highlights

* Forgotten user directive

* Fix alignment

* Cleaning up for Andy's comments

* Mention beta_loss=0 speedup

* Update

* Clean up new what's new entries

* DOC Add changes missed from what's new

And other minor things.

This took lots of effort which I would have not committed where I not home sick...
yarikoptic added a commit to yarikoptic/scikit-learn that referenced this pull request Jul 27, 2017
Release 0.19b2

* tag '0.19b2': (808 commits)
  Preparing 0.19b2
  [MRG+1] FIX out of bounds array access in SAGA (scikit-learn#9376)
  FIX make test_importances pass on 32 bit linux
  Release 0.19b1
  DOC remove 'in dev' header in whats_new.rst
  DOC typos in whats_news.rst [ci skip]
  [MRG] DOC cleaning up what's new for 0.19 (scikit-learn#9252)
  FIX t-SNE memory usage and many other optimizer issues (scikit-learn#9032)
  FIX broken link in gallery and bad title rendering
  [MRG] DOC Replace \acute by prime (scikit-learn#9332)
  Fix typos (scikit-learn#9320)
  [MRG + 1 (rv) + 1 (alex) + 1] Add a check to test the docstring params and their order (scikit-learn#9206)
  DOC Residual sum vs. regression sum (scikit-learn#9314)
  [MRG] [HOTFIX] Fix capitalization in test and hence fix failing travis at master (scikit-learn#9317)
  More informative error message for classification metrics given regression output (scikit-learn#9275)
  [MRG] COSMIT Remove unused parameters in private functions (scikit-learn#9310)
  [MRG+1] Ridgecv normalize (scikit-learn#9302)
  [MRG + 2] ENH Allow `cross_val_score`, `GridSearchCV` et al. to evaluate on multiple metrics (scikit-learn#7388)
  Add data_home parameter to fetch_kddcup99 (scikit-learn#9289)
  FIX makedirs(..., exists_ok) not available in Python 2 (scikit-learn#9284)
  ...
yarikoptic added a commit to yarikoptic/scikit-learn that referenced this pull request Jul 27, 2017
* releases: (808 commits)
  Preparing 0.19b2
  [MRG+1] FIX out of bounds array access in SAGA (scikit-learn#9376)
  FIX make test_importances pass on 32 bit linux
  Release 0.19b1
  DOC remove 'in dev' header in whats_new.rst
  DOC typos in whats_news.rst [ci skip]
  [MRG] DOC cleaning up what's new for 0.19 (scikit-learn#9252)
  FIX t-SNE memory usage and many other optimizer issues (scikit-learn#9032)
  FIX broken link in gallery and bad title rendering
  [MRG] DOC Replace \acute by prime (scikit-learn#9332)
  Fix typos (scikit-learn#9320)
  [MRG + 1 (rv) + 1 (alex) + 1] Add a check to test the docstring params and their order (scikit-learn#9206)
  DOC Residual sum vs. regression sum (scikit-learn#9314)
  [MRG] [HOTFIX] Fix capitalization in test and hence fix failing travis at master (scikit-learn#9317)
  More informative error message for classification metrics given regression output (scikit-learn#9275)
  [MRG] COSMIT Remove unused parameters in private functions (scikit-learn#9310)
  [MRG+1] Ridgecv normalize (scikit-learn#9302)
  [MRG + 2] ENH Allow `cross_val_score`, `GridSearchCV` et al. to evaluate on multiple metrics (scikit-learn#7388)
  Add data_home parameter to fetch_kddcup99 (scikit-learn#9289)
  FIX makedirs(..., exists_ok) not available in Python 2 (scikit-learn#9284)
  ...
yarikoptic added a commit to yarikoptic/scikit-learn that referenced this pull request Jul 27, 2017
* dfsg: (808 commits)
  Preparing 0.19b2
  [MRG+1] FIX out of bounds array access in SAGA (scikit-learn#9376)
  FIX make test_importances pass on 32 bit linux
  Release 0.19b1
  DOC remove 'in dev' header in whats_new.rst
  DOC typos in whats_news.rst [ci skip]
  [MRG] DOC cleaning up what's new for 0.19 (scikit-learn#9252)
  FIX t-SNE memory usage and many other optimizer issues (scikit-learn#9032)
  FIX broken link in gallery and bad title rendering
  [MRG] DOC Replace \acute by prime (scikit-learn#9332)
  Fix typos (scikit-learn#9320)
  [MRG + 1 (rv) + 1 (alex) + 1] Add a check to test the docstring params and their order (scikit-learn#9206)
  DOC Residual sum vs. regression sum (scikit-learn#9314)
  [MRG] [HOTFIX] Fix capitalization in test and hence fix failing travis at master (scikit-learn#9317)
  More informative error message for classification metrics given regression output (scikit-learn#9275)
  [MRG] COSMIT Remove unused parameters in private functions (scikit-learn#9310)
  [MRG+1] Ridgecv normalize (scikit-learn#9302)
  [MRG + 2] ENH Allow `cross_val_score`, `GridSearchCV` et al. to evaluate on multiple metrics (scikit-learn#7388)
  Add data_home parameter to fetch_kddcup99 (scikit-learn#9289)
  FIX makedirs(..., exists_ok) not available in Python 2 (scikit-learn#9284)
  ...
dmohns pushed a commit to dmohns/scikit-learn that referenced this pull request Aug 7, 2017
* DOC cleaning up what's new for 0.19

* More cleaning up

* More cleaning up

* Deprecations

* Clean up merge

* Update

* TODOs to prose and minor changes

* Changed models and minor fixes

* sort

* Merge in 0.18.2 docs

* Missing entry from 0.18 logs

* Optimistically add some features to highlights

* Forgotten user directive

* Fix alignment

* Cleaning up for Andy's comments

* Mention beta_loss=0 speedup

* Update

* Clean up new what's new entries

* DOC Add changes missed from what's new

And other minor things.

This took lots of effort which I would have not committed where I not home sick...
dmohns pushed a commit to dmohns/scikit-learn that referenced this pull request Aug 7, 2017
* DOC cleaning up what's new for 0.19

* More cleaning up

* More cleaning up

* Deprecations

* Clean up merge

* Update

* TODOs to prose and minor changes

* Changed models and minor fixes

* sort

* Merge in 0.18.2 docs

* Missing entry from 0.18 logs

* Optimistically add some features to highlights

* Forgotten user directive

* Fix alignment

* Cleaning up for Andy's comments

* Mention beta_loss=0 speedup

* Update

* Clean up new what's new entries

* DOC Add changes missed from what's new

And other minor things.

This took lots of effort which I would have not committed where I not home sick...
NelleV pushed a commit to NelleV/scikit-learn that referenced this pull request Aug 11, 2017
* DOC cleaning up what's new for 0.19

* More cleaning up

* More cleaning up

* Deprecations

* Clean up merge

* Update

* TODOs to prose and minor changes

* Changed models and minor fixes

* sort

* Merge in 0.18.2 docs

* Missing entry from 0.18 logs

* Optimistically add some features to highlights

* Forgotten user directive

* Fix alignment

* Cleaning up for Andy's comments

* Mention beta_loss=0 speedup

* Update

* Clean up new what's new entries

* DOC Add changes missed from what's new

And other minor things.

This took lots of effort which I would have not committed where I not home sick...
paulha pushed a commit to paulha/scikit-learn that referenced this pull request Aug 19, 2017
* DOC cleaning up what's new for 0.19

* More cleaning up

* More cleaning up

* Deprecations

* Clean up merge

* Update

* TODOs to prose and minor changes

* Changed models and minor fixes

* sort

* Merge in 0.18.2 docs

* Missing entry from 0.18 logs

* Optimistically add some features to highlights

* Forgotten user directive

* Fix alignment

* Cleaning up for Andy's comments

* Mention beta_loss=0 speedup

* Update

* Clean up new what's new entries

* DOC Add changes missed from what's new

And other minor things.

This took lots of effort which I would have not committed where I not home sick...
AishwaryaRK pushed a commit to AishwaryaRK/scikit-learn that referenced this pull request Aug 29, 2017
* DOC cleaning up what's new for 0.19

* More cleaning up

* More cleaning up

* Deprecations

* Clean up merge

* Update

* TODOs to prose and minor changes

* Changed models and minor fixes

* sort

* Merge in 0.18.2 docs

* Missing entry from 0.18 logs

* Optimistically add some features to highlights

* Forgotten user directive

* Fix alignment

* Cleaning up for Andy's comments

* Mention beta_loss=0 speedup

* Update

* Clean up new what's new entries

* DOC Add changes missed from what's new

And other minor things.

This took lots of effort which I would have not committed where I not home sick...
maskani-moh pushed a commit to maskani-moh/scikit-learn that referenced this pull request Nov 15, 2017
* DOC cleaning up what's new for 0.19

* More cleaning up

* More cleaning up

* Deprecations

* Clean up merge

* Update

* TODOs to prose and minor changes

* Changed models and minor fixes

* sort

* Merge in 0.18.2 docs

* Missing entry from 0.18 logs

* Optimistically add some features to highlights

* Forgotten user directive

* Fix alignment

* Cleaning up for Andy's comments

* Mention beta_loss=0 speedup

* Update

* Clean up new what's new entries

* DOC Add changes missed from what's new

And other minor things.

This took lots of effort which I would have not committed where I not home sick...
jwjohnson314 pushed a commit to jwjohnson314/scikit-learn that referenced this pull request Dec 18, 2017
* DOC cleaning up what's new for 0.19

* More cleaning up

* More cleaning up

* Deprecations

* Clean up merge

* Update

* TODOs to prose and minor changes

* Changed models and minor fixes

* sort

* Merge in 0.18.2 docs

* Missing entry from 0.18 logs

* Optimistically add some features to highlights

* Forgotten user directive

* Fix alignment

* Cleaning up for Andy's comments

* Mention beta_loss=0 speedup

* Update

* Clean up new what's new entries

* DOC Add changes missed from what's new

And other minor things.

This took lots of effort which I would have not committed where I not home sick...
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.

4 participants