Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Support sparse input for SVC and SVR #5273
Support sparse input for SVC and SVR #5273
Changes from 33 commits
cc52cc6
02ac8b4
d7ac752
97f1812
e5b39a6
b90b716
40ad64a
0e63851
cfd01b2
e4fae32
9879f97
63f47ff
b70e77d
062a118
078ba5d
a76e3bc
a0a3da5
d307995
19bd708
45ad663
72760a2
04f8b65
58fc7be
7beab5b
977fded
74d42cb
f6664ed
0d07126
b63d7d2
1e319da
ae8f176
fc5f3cb
6cbc395
622fcde
fc0f291
7385c8e
a16ead3
22a3b34
5835578
a3c0257
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we accept such a limited set of types for this, we could probably eventually use the
raft::csr_matrix_view
but raw pointers from the Python->C++ hand-off is fine too since we really have not start porting over any of our other C++ APIs to accept mdspan directly yet.