Skip to content

Conversation

sraikund16
Copy link
Contributor

@sraikund16 sraikund16 commented Jul 8, 2025

Summary: We added the ability to make Annotating Global or Local based on an input flag in PyTorch but didn't add the args to the linter

Reviewed By: mzzchy

Differential Revision: D77959409

Summary: We added the ability to make Annotating Global or Local based on an input flag in PyTorch but now we need to have this flag propogate in the plugin itself

Test Plan:
Ran snapshot with flag on and saw that the global registration happened instead of local registration

Rollback Plan:

Reviewed By: mzzchy

Differential Revision: D77959409
Copy link

pytorch-bot bot commented Jul 8, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/157858

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 0f81b34 with merge base 2e14069 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot
Copy link
Contributor

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

@sraikund16 sraikund16 changed the title [Memory Snapshot] Pass Global Annotations into Snapshot Plugin [Memory Snapshot] Fix Linter for Global Annotations flag in Snapshot Jul 8, 2025
@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Jul 8, 2025
@sraikund16 sraikund16 added release notes: profiler release notes category topic: bug fixes topic category labels Jul 8, 2025
@facebook-github-bot
Copy link
Contributor

@pytorchbot merge

(Initiating merge automatically since Phabricator Diff has merged)

@pytorchmergebot
Copy link
Collaborator

Merge started

Your 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

Advanced Debugging
Check the merge workflow status
here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk Trigger trunk jobs on your pull request fb-exported Merged release notes: profiler release notes category topic: bug fixes topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants