Skip to content
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

MAINT: Update minimum required scikit-learn version to 1.3.0 #58

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

jhlegarreta
Copy link
Contributor

Update minimum required scikit-learn version to 1.3.0 (released June 2023):

Documentation:
https://scikit-learn.org/1.6/whats_new/v1.3.html#version-1-3-0

Update minimum required `scikit-learn` version to 1.3.0 (released June
2023):
- 0.18 dates back to Sept 28, 2016 (9 years ago):
  https://scikit-learn.org/1.6/whats_new/v0.18.html#changes-0-18
- CI testing is using 1.6.0, the most recent version to date:
  https://scikit-learn.org/1.6/whats_new/v1.6.html
- Following the errors discovered during the type hinting efforts,
  the `DiffusionGPR` class is accepting the `n_targets` named parameter
  constraint, which was introduced in `scikit-learn` version 1.3.0:
  https://scikit-learn.org/1.6/whats_new/v1.3.html#sklearn-gaussian-process

Documentation:
https://scikit-learn.org/1.6/whats_new/v1.3.html#version-1-3-0
@jhlegarreta
Copy link
Contributor Author

X-ref: #28 (comment)

@oesteban oesteban changed the title ENH: Update minimum required scikit-learn version to 1.3.0 MAINT: Update minimum required scikit-learn version to 1.3.0 Jan 20, 2025
@oesteban oesteban merged commit 0236045 into nipreps:main Jan 20, 2025
7 checks passed
@jhlegarreta jhlegarreta deleted the UpdateMinReqScikitlearn branch January 20, 2025 15:23
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.

2 participants