Skip to content

Conversation

@yushangdi
Copy link
Contributor

@yushangdi yushangdi commented May 21, 2025

Update the file names used for provenance tracking due to changes in https://github.com/pytorch/pytorch/pull/153584/files#diff-e0cdb58c0f84f56f20c5433339b6d83c470dcde47847e2328effea6bedd4cd27

Additional changes:

  • update the test log files
  • Highlight the whole line
  • Allow panel size changes

@yushangdi
Copy link
Contributor Author

@angelayi @jamesjwu Can I get a review/stamp for this PR? Thank you!

Copy link

@YUNQIUGUO YUNQIUGUO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jamesjwu jamesjwu merged commit d717e4f into meta-pytorch:main May 21, 2025
15 checks passed
// Generate HTML for each directory name
for directory_name in &directory_names {
let pre_grad_graph_content =
get_file_content(&output, "inductor_pre_grad_graph", directory_name);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to keep the old names for backward compatibility.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to keep the old names for backward compatibility.

@desertfire I'll add back the file name to be BC. But I'm updating the doc to say it's only compatible with the nightly build in pytorch/pytorch#154062 because it may still not be compatible with earlier versions of PyTorch (e.g. a few months ago).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@desertfire added back in PR #111

@yushangdi yushangdi deleted the update_provenance_name branch May 22, 2025 16:52
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request May 23, 2025
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.

5 participants