Skip to content

DOC Update array API common checks docs #31050

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 4 commits into from
May 13, 2025

Conversation

betatim
Copy link
Member

@betatim betatim commented Mar 21, 2025

Follow on from discussion in #30340 (comment) - that PR should be merged first, then I can update this one to resolve problems.

This changes the docs to mention that you can run (most of the) checks without having a GPU by using array-api-strict.

cc @lucascolley

Mention that you can run (most of the) checks without having a GPU by
using array-api-strict.
Copy link

github-actions bot commented Mar 21, 2025

✔️ Linting Passed

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

Generated for commit: 58bd8df. Link to the linter CI: here

betatim and others added 2 commits May 12, 2025 10:38
Co-authored-by: Olivier Grisel <olivier.grisel@ensta.org>
@betatim
Copy link
Member Author

betatim commented May 12, 2025

I refreshed this PR that I found behind my GitHub notifications sofa :D

Let's see if we can get it merged before it falls into another crack

@lesteve
Copy link
Member

lesteve commented May 13, 2025

Let's merge this one before the sofa eats it, thanks 😉.

I have enabled auto-merge and rerun the failing CI build with the figshare issue #30761 🤞.

@lesteve lesteve enabled auto-merge (squash) May 13, 2025 12:25
@lesteve lesteve merged commit e906f0e into scikit-learn:main May 13, 2025
34 checks passed
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