adding arg values and arg types to Strobelight USDT#155185
adding arg values and arg types to Strobelight USDT#155185aasthashah999 wants to merge 1 commit intopytorch:mainfrom
Conversation
|
This appears to be a diff that was exported from phabricator, but the PR author does not have sufficient permissions to run CI. @aasthashah999, please do step 2 of internal wiki to get write access so you do not need to get CI approvals in the future. If you think this is a mistake, please contact the Pytorch Dev Infra team. |
|
|
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/155185
Note: Links to docs will display an error until the docs builds have been completed. ⏳ 1 Pending, 1 Unrelated FailureAs of commit 20effb0 with merge base b055611 ( UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
This pull request was exported from Phabricator. Differential Revision: D75606556 |
0faf62a to
d82596c
Compare
|
This pull request was exported from Phabricator. Differential Revision: D75606556 |
d82596c to
add12fe
Compare
add12fe to
73e14ab
Compare
|
This pull request was exported from Phabricator. Differential Revision: D75606556 |
73e14ab to
9ff8d95
Compare
|
@pytorchbot label "topic: not user facing" |
9ff8d95 to
8aaef9e
Compare
|
This pull request was exported from Phabricator. Differential Revision: D75606556 |
8aaef9e to
f325794
Compare
f325794 to
10dc884
Compare
|
This pull request was exported from Phabricator. Differential Revision: D75606556 |
10dc884 to
f90b6b9
Compare
|
@pytorchmergebot merge |
|
Pull workflow has not been scheduled for the PR yet. It could be because author doesn't have permissions to run those or skip-checks keywords were added to PR/commits, aborting merge. Please get/give approval for the workflows and/or remove skip ci decorators before next merge attempt. If you think this is a mistake, please contact PyTorch Dev Infra. |
8fc4d06 to
c2ff1fc
Compare
|
This pull request was exported from Phabricator. Differential Revision: D75606556 |
2058a30 to
28f341f
Compare
|
This pull request was exported from Phabricator. Differential Revision: D75606556 |
8485106 to
ea5e587
Compare
|
This pull request was exported from Phabricator. Differential Revision: D75606556 |
53f49c7 to
b62c434
Compare
Summary: Pull Request resolved: pytorch#155185 This diff makes changes to the USDT added by RihamSelim in D44636587. The "operator_start" USDT passes in the memory addresses of operator arguments and the argument types. This is so we can record argument values and types in the Strobelight GPUEvent Profiler. The previous diff records the ATEN operator, and this diff lays the groundwork to record ATEN op arguments. Test Plan: I ensured this code builds by running the example in this diff, and testing profiler changes in this diff. Reviewed By: malfet, RihamSelim Differential Revision: D75606556
|
This pull request was exported from Phabricator. Differential Revision: D75606556 |
b62c434 to
20effb0
Compare
|
@pytorchbot merge (Initiating merge automatically since Phabricator Diff has merged) |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Summary: This diff makes changes to the USDT added by RihamSelim in D44636587. The "operator_start" USDT passes in the memory addresses of operator arguments and the argument types. This is so we can record argument values and types in the Strobelight GPUEvent Profiler. The previous diff records the ATEN operator, and this diff lays the groundwork to record ATEN op arguments.
Test Plan: I ensured this code builds by running the example in this diff, and testing profiler changes in this diff.
Reviewed By: RihamSelim
Differential Revision: D75606556