-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
[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
Conversation
doc/whats_new.rst
Outdated
: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`_. |
There was a problem hiding this comment.
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".
There was a problem hiding this comment.
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?
doc/whats_new.rst
Outdated
would fail with a unmeaningful error by default. | ||
:issue:`6573` by :user:`Quazi Marufur Rahman <qmaruf>` and | ||
`Manoj Kumar`_. | ||
Linear, kernelized and related models |
There was a problem hiding this comment.
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?
I am fine to make more or different headings if it is helpful.
…On 5 Jul 2017 12:15 am, "Olivier Grisel" ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In doc/whats_new.rst
<#9252 (comment)>
:
>
- - Fixed a bug in :class:`gaussian_process.GaussianProcessRegressor`
- when the standard deviation and covariance predicted without fit
- would fail with a unmeaningful error by default.
- :issue:`6573` by :user:`Quazi Marufur Rahman <qmaruf>` and
- `Manoj Kumar`_.
+Linear, kernelized and related models
Maybe we should have one section name "Neighborhood models" and another
"Semi-supervised learning models" instead?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#9252 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAEz6xLGkDt9e2iSoj3KyrSfjZRW0y0Jks5sKkkGgaJpZM4OKFjl>
.
|
of course could just group by subpackage
…On 5 Jul 2017 7:39 am, "Joel Nothman" ***@***.***> wrote:
I am fine to make more or different headings if it is helpful.
On 5 Jul 2017 12:15 am, "Olivier Grisel" ***@***.***> wrote:
> ***@***.**** commented on this pull request.
> ------------------------------
>
> In doc/whats_new.rst
> <#9252 (comment)>
> :
>
> >
> - - Fixed a bug in :class:`gaussian_process.GaussianProcessRegressor`
> - when the standard deviation and covariance predicted without fit
> - would fail with a unmeaningful error by default.
> - :issue:`6573` by :user:`Quazi Marufur Rahman <qmaruf>` and
> - `Manoj Kumar`_.
> +Linear, kernelized and related models
>
> Maybe we should have one section name "Neighborhood models" and another
> "Semi-supervised learning models" instead?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <#9252 (review)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAEz6xLGkDt9e2iSoj3KyrSfjZRW0y0Jks5sKkkGgaJpZM4OKFjl>
> .
>
|
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? |
can you merge it with the 0.18.2 whatsnew? |
doc/whats_new.rst
Outdated
- 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 |
There was a problem hiding this comment.
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?
I push a couple of changes but apart from the empty "Metrics" section, LGTM. |
There was a problem hiding this 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 |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
doc/whats_new.rst
Outdated
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 |
There was a problem hiding this comment.
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?
doc/whats_new.rst
Outdated
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. |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
doc/whats_new.rst
Outdated
: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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LabelPropagation?
doc/whats_new.rst
Outdated
- 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wast not?
doc/whats_new.rst
Outdated
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
double backticks?
doc/whats_new.rst
Outdated
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
double backtricks everywhere?
doc/whats_new.rst
Outdated
- 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_` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
double backticks?
doc/whats_new.rst
Outdated
: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`_. |
There was a problem hiding this comment.
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?
Maybe we've missed something? Make sure you go back and check the logs before giving this LGTM. :P |
why are there conflicts in the tsne test? |
Weird. Must have been a merge error on my part. I'll fix it up shortly |
Marking MRG |
And other minor things. This took lots of effort which I would have not committed where I not home sick...
Okay. Let's do a release, then :) |
* 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...
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) ...
* 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) ...
* 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) ...
* 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...
* 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...
* 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...
* 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...
* 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...
* 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...
* 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...
subsections, reclassifications, etc
Marked WIP only because of optimistic inclusion of multiple metrics, which is not yet merged.