Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Docs: Add Nvidia RAPIDS#31682

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
smith558 wants to merge1 commit intoscikit-learn:main
base:main
Choose a base branch
Loading
fromsmith558:patch-2

Conversation

smith558
Copy link
Contributor

Reference Issues/PRs

What does this implement/fix? Explain your changes.

Add Nvidia RAPIDS

Any other comments?

@github-actionsGitHub Actions
Copy link

✔️ Linting Passed

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

Generated for commit:92afcee. Link to the linter CI:here

Copy link
Member

@virchanvirchan left a comment

Choose a reason for hiding this comment

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

Thank you for the PR,@smith558!

@adrinjalali, just a friendly ping, do you think NVIDIA RAPIDS would be appropriate to include on the"Related Projects" page?

@ogrisel
Copy link
Member

+1 for referencing RAPIDS in that section, but the disclaimer for the scikit-learn-intelex item should be extracted as a new paragraph at the beginning of the "Model throughput" section because it would apply for both projects.

@adrinjalali
Copy link
Member

I'm not sure about this one. Should it not instead be the cuml project?

ogrisel reacted with thumbs up emoji

@smith558
Copy link
ContributorAuthor

I'm not sure about this one. Should it not instead be the cuml project?

Indeed. It is cuML. But cuML is a part of RAPIDS. Not a separate project.

@adrinjalali
Copy link
Member

Indeed. It is cuML. But cuML is a part of RAPIDS. Not a separate project.

And intelex is a part of intel, that doesn't mean we refer to intel as a related project 😅

From the perspective of our users, and us, cuML is a separate project, separate repo, separatepypi package. It's as if you reference PyData packages or include numpy/scipy/scikit-learn all as the same "project".

lorentzenchr and betatim reacted with thumbs up emojismith558 reacted with thumbs down emoji

@smith558
Copy link
ContributorAuthor

smith558 commentedJul 2, 2025
edited
Loading

And intelex is a part of intel, that doesn't mean we refer to intel as a related project 😅

Sorry, the project is literally calledRAPIDS, it's the preferred way of the maintainers. When were we discussing the word "Nvidia"? My comment said:

Indeed. It is cuML. But cuML is a part of RAPIDS. Not a separate project.

I am happy for you to drop "Nvidia". (Even though I think it makes sense to keep "Nvidia" in it, because it's strictly only for Nvidia chips.)

And intelex is a part of intel, that doesn't mean we refer to intel as a related project 😅

No, you do. In thedocs you literally callintelex "Intel(R) Extension for scikit-learn". (as shown below)

image

@adrinjalali
Copy link
Member

I'd be okay with includingcuML as the main item, but mentioning it's a part of the RAPIDS ecosystem.

@ogrisel
Copy link
Member

ogrisel commentedJul 10, 2025
edited
Loading

Why not refer to "RAPIDS cuML" to be maximally informative?

I agree with@adrinjalali that we should be specific about which sub-package of the RAPIDS ecosystem is relevant to accelerate scikit-learn on nvidia hardware.

betatim reacted with thumbs up emoji

@betatim
Copy link
Member

👍 to the suggestion of referring directly to cuml. Otherwise you need to explain to people that they don't need to install all of RAPIDS but only the cuml part. Of course you could install of RAPIDS to get cuml, but it is not needed. IMHO it is a bit like telling people to install anaconda (the distribution) in order to use scikit-learn. (I work for NVIDIA as part of the RAPIDS team - jupp RAPIDS is an overloaded term :D)

virchan reacted with thumbs up emojiadrinjalali reacted with laugh emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@virchanvirchanvirchan left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

5 participants
@smith558@ogrisel@adrinjalali@betatim@virchan

[8]ページ先頭

©2009-2025 Movatter.jp