Skip to content

🔒 🤖 CI Update lock files for main CI build(s) 🔒 🤖#30593

Merged
jeremiedbb merged 3 commits into
scikit-learn:mainfrom
scikit-learn-bot:auto-update-lock-files-main
Jan 6, 2025
Merged

🔒 🤖 CI Update lock files for main CI build(s) 🔒 🤖#30593
jeremiedbb merged 3 commits into
scikit-learn:mainfrom
scikit-learn-bot:auto-update-lock-files-main

Conversation

@scikit-learn-bot

Copy link
Copy Markdown
Contributor

Update lock files.

Note

If the CI tasks fail, create a new branch based on this PR and add the required fixes to that branch.

@github-actions

github-actions Bot commented Jan 6, 2025

Copy link
Copy Markdown

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 5687956. Link to the linter CI: here

@ogrisel

ogrisel commented Jan 6, 2025

Copy link
Copy Markdown
Member

The failure is weird:

________________________ [doctest] common_pitfalls.rst _________________________
218     0.46
219 
220 The pipeline can also be fed into a cross-validation
221 function such as :func:`~sklearn.model_selection.cross_val_score`.
222 Again, the pipeline ensures that the correct data subset and estimator
223 method is used during fitting and predicting::
224 
225     >>> from sklearn.model_selection import cross_val_score
226     >>> scores = cross_val_score(pipeline, X, y)
227     >>> print(f"Mean accuracy: {scores.mean():.2f}+/-{scores.std():.2f}")
Expected:
    Mean accuracy: 0.46+/-0.07
Got:
    Mean accuracy: 0.45+/-0.07

It might be related to the upgrade to scipy 1.15.0

@jeremiedbb why do you think that merging main would help?

@jeremiedbb

Copy link
Copy Markdown
Member

because I just merged #30583 which fixes it :)

@jeremiedbb jeremiedbb merged commit 51ca364 into scikit-learn:main Jan 6, 2025
jeremiedbb added a commit to jeremiedbb/scikit-learn that referenced this pull request Jan 8, 2025
Co-authored-by: Lock file bot <noreply@github.com>
Co-authored-by: Jérémie du Boisberranger <jeremie@probabl.ai>
jeremiedbb added a commit to jeremiedbb/scikit-learn that referenced this pull request Jan 8, 2025
Co-authored-by: Lock file bot <noreply@github.com>
Co-authored-by: Jérémie du Boisberranger <jeremie@probabl.ai>
jeremiedbb added a commit that referenced this pull request Jan 9, 2025
Co-authored-by: Lock file bot <noreply@github.com>
Co-authored-by: Jérémie du Boisberranger <jeremie@probabl.ai>
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