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.

Sign up for GitHub

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

Jump to bottom

[REVIEW] Add NVTX ranges to RMM #336

Closed
wants to merge 9 commits into from
Closed

[REVIEW] Add NVTX ranges to RMM #336

wants to merge 9 commits into from

Conversation

harrism
Copy link
Member

@harrism harrism commented Mar 18, 2020

As in libcudf, adds an RMM detail/nvtx/ranges.hpp with some convenience functions for defining NVTX ranges in RMM functions. Also adds ranges to the various memory_resource classes.

@harrism harrism added the 3 - Ready for review Ready for review by team label Mar 18, 2020
@harrism harrism requested a review from jrhemstad March 18, 2020 01:32
@harrism harrism requested a review from a team as a code owner March 18, 2020 01:32
@harrism harrism self-assigned this Mar 18, 2020
@harrism harrism added this to PR-WIP in v0.13 Release via automation Mar 18, 2020
@harrism harrism changed the title Add NVTX ranges to RMM [REVIEW] Add NVTX ranges to RMM Mar 18, 2020
@harrism harrism moved this from PR-WIP to PR-Needs review in v0.13 Release Mar 18, 2020
@harrism harrism changed the base branch from branch-0.13 to branch-0.14 March 26, 2020 21:18
@harrism harrism added this to PR-WIP in v0.14 Release via automation Mar 26, 2020
@harrism harrism removed this from PR-Needs review in v0.13 Release Mar 26, 2020
@harrism harrism removed this from PR-WIP in v0.14 Release May 28, 2020
@harrism harrism added this to PR-WIP in v0.15 Release via automation May 28, 2020
@harrism harrism changed the base branch from branch-0.14 to branch-0.15 May 28, 2020 00:31
@harrism harrism removed this from PR-WIP in v0.15 Release Aug 7, 2020
@harrism harrism added this to PR-WIP in v0.16 Release via automation Aug 7, 2020
@jrhemstad jrhemstad mentioned this pull request Aug 14, 2020
Closed
@harrism harrism changed the base branch from branch-0.15 to branch-0.16 August 24, 2020 01:55
@harrism harrism requested a review from a team as a code owner August 24, 2020 01:55
@harrism harrism added this to PR-WIP in v0.17 Release via automation Sep 29, 2020
@harrism harrism removed this from PR-WIP in v0.16 Release Sep 29, 2020
@harrism harrism changed the base branch from branch-0.16 to branch-0.17 October 29, 2020 07:07
@harrism harrism requested a review from a team as a code owner October 29, 2020 07:07
@github-actions
Copy link

This PR has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates. This PR will be labeled inactive-90d if there is no activity in the next 60 days.

@kkraus14 kkraus14 added 0 - Backlog In queue waiting for assignment and removed 3 - Ready for review Ready for review by team inactive-30d labels Mar 18, 2021
@harrism harrism added this to PR-WIP in v21.06 Release via automation Apr 19, 2021
@harrism harrism removed this from PR-WIP in v0.19 Release Apr 19, 2021
@harrism harrism removed this from PR-WIP in v21.06 Release May 25, 2021
@harrism harrism added this to PR-WIP in v21.08 Release via automation May 25, 2021
@harrism harrism removed this from PR-WIP in v21.08 Release Jul 20, 2021
@harrism harrism added this to PR-WIP in v21.10 Release via automation Jul 20, 2021
@harrism harrism removed this from PR-WIP in v21.10 Release Jul 20, 2021
@harrism harrism added this to PR-WIP in v21.08 Release via automation Jul 20, 2021
@harrism harrism removed this from PR-WIP in v21.08 Release Jul 20, 2021
@harrism harrism added this to PR-WIP in v21.10 Release via automation Jul 20, 2021
@harrism harrism removed this from PR-WIP in v21.10 Release Jul 20, 2021
@harrism harrism added this to PR-WIP in v21.08 Release via automation Jul 20, 2021
@harrism harrism removed this from PR-WIP in v21.08 Release Jul 20, 2021
@harrism harrism added this to PR-WIP in v21.10 Release via automation Jul 20, 2021
@harrism harrism removed this from PR-WIP in v21.10 Release Sep 16, 2021
@harrism harrism added this to PR-WIP in v21.12 Release via automation Sep 16, 2021
@harrism harrism removed this from PR-WIP in v21.12 Release Nov 9, 2021
@harrism harrism added this to PR-WIP in v22.02 Release via automation Nov 9, 2021
@harrism harrism removed this from PR-WIP in v22.02 Release Jan 11, 2022
@harrism harrism added this to PR-WIP in v22.04 Release via automation Jan 11, 2022
@harrism harrism removed this from PR-WIP in v22.04 Release Mar 14, 2022
@harrism
Copy link
Member Author

harrism commented May 22, 2024

Superseded by #1558

@harrism harrism closed this May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jrhemstad jrhemstad jrhemstad requested changes

@rongou rongou Awaiting requested review from rongou

Assignees

@harrism harrism

Labels
0 - Backlog In queue waiting for assignment
Projects
RMM Project Board
Status: Done
Feature Planning
Awaiting triage
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@harrism @jrhemstad @kkraus14