Skip to content
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

[FEA]: Use RAFT get/set_current_device_resource() wrappers in cuGraph #4643

Open
2 tasks done
harrism opened this issue Sep 4, 2024 · 0 comments · May be fixed by #4644
Open
2 tasks done

[FEA]: Use RAFT get/set_current_device_resource() wrappers in cuGraph #4643

harrism opened this issue Sep 4, 2024 · 0 comments · May be fixed by #4644
Assignees
Labels
feature request New feature or request

Comments

@harrism
Copy link
Member

harrism commented Sep 4, 2024

Is this a new feature, an improvement, or a change to existing functionality?

New Feature

How would you describe the priority of this feature request

High

Please provide a clear description of problem this feature solves

To reduce thrash due to RMM refactoring, RAFT is adding wrappers around RMM rmm::get_current_device_resource(), rmm::set_current_device_resource() and aliases for RMM resource_ref types.

See rapidsai/raft#2424

Using these will allow us to change them in a single location in RAFT without having to touch every file that uses memory resources.

Describe your ideal solution

See above

Describe any alternatives you have considered

None

Additional context

No response

Code of Conduct

  • I agree to follow cuGraph's Code of Conduct
  • I have searched the open feature requests and have found no duplicates for this feature request
@harrism harrism added ? - Needs Triage Need team to review and classify feature request New feature or request labels Sep 4, 2024
@harrism harrism self-assigned this Sep 4, 2024
@harrism harrism removed the ? - Needs Triage Need team to review and classify label Sep 4, 2024
@harrism harrism linked a pull request Sep 4, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant