Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

NVTX3 header library integration - Change of user contrallable NVTX domains to categories #17

Draft
wants to merge 29 commits into
base: master
Choose a base branch
from
Draft

NVTX3 header library integration - Change of user contrallable NVTX domains to categories #17

wants to merge 29 commits into from

Conversation

DEKHTIARJonathan
Copy link
Collaborator

@DEKHTIARJonathan DEKHTIARJonathan commented Apr 10, 2020 •
edited
Loading

Changes in this PR:

  • Use of NVTXv3 header only library
  • Removed user controllable NVTX domains in favor of NVTX categories
  • Remove mentions of NVTX domains
  • Add deprecated decorators

@DEKHTIARJonathan DEKHTIARJonathan added enhancement New feature or request WIP labels Apr 10, 2020
@DEKHTIARJonathan DEKHTIARJonathan added this to the 0.1.9 milestone Apr 10, 2020
@DEKHTIARJonathan DEKHTIARJonathan self-assigned this Apr 10, 2020
@DEKHTIARJonathan DEKHTIARJonathan marked this pull request as draft April 10, 2020 19:37
@lgtm-com
Copy link

lgtm-com bot commented May 5, 2020

This pull request introduces 1 alert when merging 945263d into 848f98c - view on LGTM.com

new alerts:

  • 1 for Unnecessary pass

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@DEKHTIARJonathan DEKHTIARJonathan

Labels
enhancement New feature or request WIP
Projects
None yet
Milestone
0.1.9
Development

Successfully merging this pull request may close these issues.
1 participant
@DEKHTIARJonathan