8000 Release 0.20.2 by amueller · Pull Request #12784 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]

Skip to content

Release 0.20.2 #12784

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 41 commits into from
Dec 18, 2018
Merged

Release 0.20.2 #12784

merged 41 commits into from
Dec 18, 2018

Conversation

amueller
Copy link
Member
@amueller amueller commented Dec 14, 2018

Doesn't include #12659 which I think was added to 0.20.2 by mistake.
Still missing #12701.

Requires adding contributors and release date as well.

@amueller amueller changed the base branch from master to 0.20.X December 14, 2018 16:48
@amueller amueller closed this Dec 14, 2018
@amueller amueller reopened this Dec 14, 2018
@qinhanmin2014
Copy link
Member

@amueller Circle CI is not running
101ddb3
3cb4a52 shouldn't be here
These are hot fixes after we remove CI < python 3.5

@amueller
Copy link
Member Author

@jnothman maybe you can tell me how you're doing the rebase now ;)
I tried running the same rebase again but looks like everything that I dropped shows up again as pick.

qinhanmin2014 and others added 24 commits December 17, 2018 15:26
The data comes from static CSVs that cover 2003-01-01 to 2007-12-31.
* fixed randomness of test_logreg_cv_penalty

* Added comment about warm-starting coefficients
albertcthomas and others added 6 commits December 17, 2018 15:26
* Update for Logistic-Regression Table Summary for L-BFGS

* Update for linear_model.rst Logistic-Regression L-BFGS summary.

* Update for Logistic-Regression Doc.

* Logistic-Regression Update

* Update linear_model.rst

* Update jnothman comments

* Update jnothman comments

* Update for jnothman comments.

* test

* test2

* test3

* test4

* Update for Comments from amueller and jnothman

* Update for Comments from amueller and jnothman
@amueller
Copy link
Member Author

Good to go?
#12671 (which was a major driver for doing another release) will hopefully be fixed as part of the release process.

@jnothman
Copy link
Member

You've not updated __version__

@jnothman
Copy link
Member

I tried running the same rebase again but looks like everything that I dropped shows up again as pick.

Yes, it does, but it's all at the beginning of the rebase list and so easy to change to drop in one go.

Copy link
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.

FYI, I've opened #12810

@qinhanmin2014
Copy link
Member

And we need to add 5ab850a to master.

@h6197627
Copy link

Hi,
#12676 won't be fixed in this release?

@jnothman
Copy link
Member

@h6197627 it's impossible to fix every bug in some release. It's also difficult to fix a bug if we can't reproduce it. I suspect the answer is no, #12676 won't be fixed until we can reproduce and diagnose it. 0.20 was also so big a release that it's hard to even guess what might have caused this (except that #10071 appears to be the only change directly touching OMP)

@jnothman
Copy link
Member

And we need to add 5ab850a to master.

Yes, ideally this would be pushed to master first

@qinhanmin2014
Copy link
Member

@jnothman I guess we need #12810 ? (and maybe #12816 ?)

@jnothman jnothman merged commit a941c2e into scikit-learn:0.20.X Dec 18, 2018
@jnothman
Copy link
Member

merging for release

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.

0