Skip to content

Add interface to register activity profilers #309

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

Closed
wants to merge 1 commit into from

Conversation

briancoutinho
Copy link
Contributor

Summary:
First cut for api to register profilers

  • Moved the profiler to a unique_ptr so it is owned by ActivityProfiler object
  • Use of factory style of initializers on libkineto API

Reviewed By: gdankel

Differential Revision: D28418449

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D28418449

Summary:
Pull Request resolved: pytorch#309

First cut for api to register profilers
* Moved the profiler to a unique_ptr so it is owned by ActivityProfiler object
* Use of factory style of initializers on libkineto API

Reviewed By: gdankel

Differential Revision: D28418449

fbshipit-source-id: 5248f40a165e21b5f50dc72cd0393e22e167d90b
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D28418449

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in ef1d450.

@facebook-github-bot
Copy link
Contributor

This pull request has been reverted by ba8bdff.

gdankel pushed a commit to gdankel/kineto that referenced this pull request Jun 25, 2021
Summary:
Pull Request resolved: pytorch#309

First cut for api to register profilers
* Moved the profiler to a unique_ptr so it is owned by ActivityProfiler object
* Use of factory style of initializers on libkineto API

Reviewed By: gdankel

Differential Revision: D28418449

fbshipit-source-id: b0f1f377a2ea344c3da681f04c14aa6e50e2d77b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants