Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BE] Change _marked_safe_globals_list to set #139303

Conversation

Copy link

pytorch-bot bot commented Oct 30, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit b008448 with merge base 73fde0d (image):
💚 Looks good so far! There are no failures yet. 💚

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

mikaylagawarecki added a commit that referenced this pull request Oct 30, 2024
ghstack-source-id: ecbe4abb73ac5c79e02152845c030386812f2166
Pull Request resolved: #139303
mikaylagawarecki added a commit that referenced this pull request Oct 30, 2024
ghstack-source-id: 1143495135b606504f2a749fdd8a0f040712f0b8
Pull Request resolved: #139303
mikaylagawarecki added a commit that referenced this pull request Oct 30, 2024
ghstack-source-id: fb6183a0733be1aa68c6a663609580f93235c7f4
Pull Request resolved: #139303
mikaylagawarecki added a commit that referenced this pull request Oct 30, 2024
ghstack-source-id: 18fe8505cda6d6e626f12c88314a9da2fa6016c7
Pull Request resolved: #139303
mikaylagawarecki added a commit that referenced this pull request Oct 30, 2024
ghstack-source-id: da7ef54da4992c85c80251906ab66056520ca501
Pull Request resolved: #139303
mikaylagawarecki added a commit that referenced this pull request Oct 31, 2024
ghstack-source-id: 88825ca646efa5bb626ae7be4acf751fef81bba2
Pull Request resolved: #139303
Prevent same global from being added multiple times




[ghstack-poisoned]
mikaylagawarecki added a commit that referenced this pull request Oct 31, 2024
ghstack-source-id: f1dd13765bfd55be91f0a86cb7d0df91c760cd78
Pull Request resolved: #139303
Prevent same global from being added multiple times




[ghstack-poisoned]
mikaylagawarecki added a commit that referenced this pull request Oct 31, 2024
ghstack-source-id: da2b06d3d0ac33d0619096663573f9bbcdb271be
Pull Request resolved: #139303
Prevent same global from being added multiple times




[ghstack-poisoned]
mikaylagawarecki added a commit that referenced this pull request Oct 31, 2024
ghstack-source-id: c7e4876e6e58f1928efae4a207610c7ced3aebdf
Pull Request resolved: #139303
Prevent same global from being added multiple times




[ghstack-poisoned]
mikaylagawarecki added a commit that referenced this pull request Oct 31, 2024
ghstack-source-id: 84c92d8be51ef6af1f1347be574bce56e1e5b45e
Pull Request resolved: #139303
Prevent same global from being added multiple times




[ghstack-poisoned]
mikaylagawarecki added a commit that referenced this pull request Oct 31, 2024
ghstack-source-id: 2214d0f025b270f2b26188d0de7926f9504fbccf
Pull Request resolved: #139303
Prevent same global from being added multiple times




[ghstack-poisoned]
mikaylagawarecki added a commit that referenced this pull request Oct 31, 2024
ghstack-source-id: 5ac797d7b5dfbabc014603f6ddf3eb83bfaf61e4
Pull Request resolved: #139303
Prevent same global from being added multiple times




[ghstack-poisoned]
mikaylagawarecki added a commit that referenced this pull request Nov 1, 2024
ghstack-source-id: 397e1b1298dd55524affc721a4b69747c1023f92
Pull Request resolved: #139303
Prevent same global from being added multiple times




[ghstack-poisoned]
mikaylagawarecki added a commit that referenced this pull request Nov 1, 2024
ghstack-source-id: 9ee1a7becfe9a062bd7253895b1591cb0bdbb3c0
Pull Request resolved: #139303
Prevent same global from being added multiple times




[ghstack-poisoned]
mikaylagawarecki added a commit that referenced this pull request Nov 1, 2024
ghstack-source-id: df14b7352246fc7882d169ff3f91728a1872b8ea
Pull Request resolved: #139303
Prevent same global from being added multiple times




[ghstack-poisoned]
mikaylagawarecki added a commit that referenced this pull request Nov 2, 2024
ghstack-source-id: 6a59b4eac5c02d657a2147f8a765cdf6e08d8e95
Pull Request resolved: #139303
Copy link
Contributor

@janeyx99 janeyx99 left a comment

Choose a reason for hiding this comment

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

lgtm

@mikaylagawarecki mikaylagawarecki marked this pull request as ready for review November 4, 2024 20:59
@mikaylagawarecki
Copy link
Contributor Author

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Nov 4, 2024
@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

atalman pushed a commit to atalman/pytorch that referenced this pull request Nov 11, 2024
Prevent same global from being added multiple times

Pull Request resolved: pytorch#139303
Approved by: https://github.com/janeyx99
ghstack dependencies: pytorch#138936, pytorch#139221, pytorch#139433, pytorch#139541, pytorch#137602
Ryo-not-rio pushed a commit to Ryo-not-rio/pytorch that referenced this pull request Dec 2, 2024
Prevent same global from being added multiple times

Pull Request resolved: pytorch#139303
Approved by: https://github.com/janeyx99
ghstack dependencies: pytorch#138936, pytorch#139221, pytorch#139433, pytorch#139541, pytorch#137602
@github-actions github-actions bot deleted the gh/mikaylagawarecki/282/head branch December 5, 2024 02:14
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 Merged topic: not user facing topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants