Minor fix to reenable nvtx sequence numbers for the forward methods of custom (Python) autograd functions#13876
Closed
mcarilli wants to merge 2 commits into
Closed
Minor fix to reenable nvtx sequence numbers for the forward methods of custom (Python) autograd functions#13876mcarilli wants to merge 2 commits into
mcarilli wants to merge 2 commits into
Conversation
… custom (Python) autograd functions
ezyang
approved these changes
Nov 13, 2018
Contributor
ezyang
left a comment
There was a problem hiding this comment.
No chance we could get a test, could we?
apaszke
approved these changes
Nov 13, 2018
Contributor
facebook-github-bot
left a comment
There was a problem hiding this comment.
@ezyang is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Collaborator
Author
|
@ezyang I'm not sure how. A test would have to do something like run nvprof on a standalone file to completion, spit out an nvvp dump, then open it (separately) and parse the sqlite database. Any suggestions? |
Contributor
|
That would probably be the most likely way to work. Maybe not worth the effort. But I promise you we will accidentally break it without any tests at some point in the future, if someone else touches this code ;) |
laurentdupin
pushed a commit
to laurentdupin/pytorch
that referenced
this pull request
Apr 24, 2026
…f custom (Python) autograd functions (pytorch#13876) Summary: Some of our arch people (mkolod, Aditya Agrawal, kevinstephano) notified me that the sequence number annotations weren't showing up for forward methods of custom autograd functions, which was breaking their nvprof dump parsing. Two one-line fixes in the appropriate code paths. Pull Request resolved: pytorch#13876 Differential Revision: D13042381 Pulled By: ezyang fbshipit-source-id: a114118f5c07ad4ba482e7a4892d08805b23c65b
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
Some of our arch people (@mkolod, Aditya Agrawal, @kevinstephano) notified me that the sequence number annotations weren't showing up for forward methods of custom autograd functions, which was breaking their nvprof dump parsing. Two one-line fixes in the appropriate code paths.