Skip to content

MNT bumb version number to 1.1 #20945

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 3 commits into from
Sep 6, 2021
Merged

Conversation

adrinjalali
Copy link
Member

Bumps the version number in main to 1.1, and adds a new what's new file.

@adrinjalali adrinjalali added this to the 1.0 milestone Sep 5, 2021
@adrinjalali
Copy link
Member Author

@glemaitre @rth maybe?

Copy link
Member

@glemaitre glemaitre left a comment

Choose a reason for hiding this comment

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

LGTM

--------------------

Version 1.1.0 of scikit-learn requires python 3.7+, numpy 1.14.6+ and
scipy 1.1.0+. Optional minimal dependency is matplotlib 2.2.2+.
Copy link
Member

Choose a reason for hiding this comment

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

do we only state matplotlib or pandas and co. to be able to run the examples?

Copy link
Member

Choose a reason for hiding this comment

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

Technically all those are optional dependencies, so it's unclear where we put the limit. I'm fine with the current version and also fine with adding pandas.

Copy link
Member Author

Choose a reason for hiding this comment

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

I also thought about that, but as far as it goes for the library, and not the examples, we only consume pandas, whereas we actually use matplotlib. So I figured I'm happy with having matplotlib here but not pandas. Once we give pandas as output, then we'd need to have it here.

Copy link
Member

@rth rth left a comment

Choose a reason for hiding this comment

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

Thanks for managing the release @adrinjalali !

--------------------

Version 1.1.0 of scikit-learn requires python 3.7+, numpy 1.14.6+ and
scipy 1.1.0+. Optional minimal dependency is matplotlib 2.2.2+.
Copy link
Member

Choose a reason for hiding this comment

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

Technically all those are optional dependencies, so it's unclear where we put the limit. I'm fine with the current version and also fine with adding pandas.

@glemaitre glemaitre mentioned this pull request Sep 6, 2021
Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
@adrinjalali
Copy link
Member Author

Merging with two approvals and green tests

@adrinjalali adrinjalali merged commit 2bbd319 into scikit-learn:main Sep 6, 2021
@adrinjalali adrinjalali deleted the version branch September 6, 2021 12:34
@glemaitre glemaitre mentioned this pull request Oct 23, 2021
10 tasks
samronsin pushed a commit to samronsin/scikit-learn that referenced this pull request Nov 30, 2021
* MNT bumb version number

* add whats new to toc tree

* Update doc/whats_new/v1.1.rst

Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>

Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
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.

3 participants