Fix get_unsafe_globals_in_checkpoint to account for user allowed globals per docstring #140738
Conversation
…als per docstring [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/140738
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ✅ No FailuresAs of commit dddee44 with merge base ebeab26 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@mikaylagawarecki has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@mikaylagawarecki has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
1 similar comment
|
@mikaylagawarecki has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
mikaylagawarecki
added
topic: bug fixes
mikaylagawarecki
added
release notes: python_frontend
|
@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 |
…als per docstring (pytorch#140738) bugfix: this function did not account for the user allowed globals :( Differential Revision: [D65960696](https://our.internmc.facebook.com/intern/diff/D65960696) Pull Request resolved: pytorch#140738 Approved by: https://github.com/malfet
…als per docstring (pytorch#140738) bugfix: this function did not account for the user allowed globals :( Differential Revision: [D65960696](https://our.internmc.facebook.com/intern/diff/D65960696) Pull Request resolved: pytorch#140738 Approved by: https://github.com/malfet
…als per docstring ghstack-source-id: c50d91f Pull Request resolved: pytorch/pytorch#140738
3 participants
bugfix: this function did not account for the user allowed globals :(
Stack from ghstack (oldest at bottom):
Differential Revision: D65960696
cc @albanD