Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/152211
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 65095c3 with merge base 1d77280 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
59b94b9 to
f310ba6
Compare
|
/easycla |
f310ba6 to
6a18143
Compare
6a18143 to
b8fb410
Compare
|
|
||
| logging | ||
| torch.compiler_aot_inductor_minifier | ||
| torch.compiler_inductor_provenance |
There was a problem hiding this comment.
This is not just an AOTInductor debugging tool, right? It is also a torch.compile debugging tool
| TorchInductor and AOTInductor Provenance Tracking | ||
| ================================================= | ||
|
|
||
| **Author:** `Shangdi Yu <https://github.com/yushangdi>`__ and `Rachel Guo <https://github.com/YUNQIUGUO>`__ |
There was a problem hiding this comment.
if you plan to keep this section here - can you please remove the Author line - we typically add this in the tutorials repo but not in regular doc sections.
There was a problem hiding this comment.
I'll move it to the tutorial repo.
| .. image:: _static/img/inductor_provenance/provenance_aot_inductor.png | ||
|
|
||
|
|
||
| Get Started |
There was a problem hiding this comment.
A more detailed heading would be better I think
| Get Started | |
| Setting up the provenance tracking |
| .. raw:: html | ||
|
|
||
| <style> | ||
| .yellow {background-color:#FFFF00;} |
There was a problem hiding this comment.
just be mindful how this will look in the dark mode: https://docs-preview.pytorch.org/pytorch/pytorch/152211/torch.compiler_inductor_provenance.html
eellison
left a comment
There was a problem hiding this comment.
thanks for doing this !
svekars
left a comment
There was a problem hiding this comment.
Just a few editorial suggestions - let me know if you have any questions.
|
@pytorchbot rebase |
|
@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here |
|
Successfully rebased |
f53ec3a to
ebde3cd
Compare
svekars
left a comment
There was a problem hiding this comment.
Really minor nits - otherwise LGTM from the publishing perspective. Someone with more technical expertise, needs to approve as well.
svekars
left a comment
There was a problem hiding this comment.
Really minor nits - otherwise LGTM from the publishing perspective. Someone with more technical expertise, needs to approve as well.
|
@pytorchbot merge |
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 |
as title