Skip to content

🔒 🤖 CI Update lock files for array-api CI build(s) 🔒 🤖 #29219

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

Conversation

scikit-learn-bot
Copy link
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.

@scikit-learn-bot scikit-learn-bot force-pushed the auto-update-lock-files-array-api branch from 9473012 to e63ab60 Compare June 10, 2024 05:03
Copy link

github-actions bot commented Jun 10, 2024

✔️ Linting Passed

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

Generated for commit: 9d98f09. Link to the linter CI: here

@ogrisel
Copy link
Member

ogrisel commented Jun 10, 2024

For information the update lock files scheduled job successfully triggered the additional tests for this PR after creating it:

but the triggered CUDA GPU CI failed for some unknown reason:

Let me trigger a new run via the workflow_dispatch form to check if this is reproducible or not:

@ogrisel
Copy link
Member

ogrisel commented Jun 10, 2024

It failed again and I can also locally reproduce that https://github.com/actions/python-versions/releases/download/3.12.4-9442300799/python-3.12.4-linux-22.04-x64.tar.gz is not a valid gzipped tar archive.

@ogrisel
Copy link
Member

ogrisel commented Jun 10, 2024

The problem was already reported upstream at:

actions/setup-python#886

@scikit-learn-bot scikit-learn-bot force-pushed the auto-update-lock-files-array-api branch from e63ab60 to 9d98f09 Compare June 10, 2024 09:17
@ogrisel
Copy link
Member

ogrisel commented Jun 10, 2024

I triggered a new CUDA GPU run for this PR at:

Copy link
Member

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

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

LGTM now that the CUDA GPU tests successfully passed on this PR.

@ogrisel ogrisel merged commit 9337a1a into scikit-learn:main Jun 10, 2024
30 checks passed
@jeremiedbb jeremiedbb mentioned this pull request Jul 2, 2024
11 tasks
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