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
fix dict size change during iteration #111267
Conversation
|
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/111267
Note: Links to docs will display an error until the docs builds have been completed. ✅ You can merge normally! (1 Unrelated Failure)As of commit ff7cdaa with merge base a063238 (): FLAKY - The following job failed but was likely due to flakiness present on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This pull request was exported from Phabricator. Differential Revision: D50283983 |
052657c
to
ac294bf
Compare
This pull request was exported from Phabricator. Differential Revision: D50283983 |
Summary: _wrapped_fns_to_patch points to f_globals which might change during iteration. This diff fixes potential runtime errors like: ``` RuntimeError: dictionary changed size during iteration ``` Test Plan: CI Reviewed By: Kronuz Differential Revision: D50283983
/easycla |
1 similar comment
/easycla |
This pull request was exported from Phabricator. Differential Revision: D50283983 |
ac294bf
to
d9bff02
Compare
This pull request was exported from Phabricator. Differential Revision: D50283983 |
Summary: _wrapped_fns_to_patch points to f_globals which might change during iteration. This diff fixes potential runtime errors like: ``` RuntimeError: dictionary changed size during iteration ``` Test Plan: CI Reviewed By: Kronuz Differential Revision: D50283983
Summary: _wrapped_fns_to_patch points to f_globals which might change during iteration. This diff fixes potential runtime errors like: ``` RuntimeError: dictionary changed size during iteration ``` Test Plan: CI Reviewed By: Kronuz Differential Revision: D50283983
This pull request was exported from Phabricator. Differential Revision: D50283983 |
d9bff02
to
ff7cdaa
Compare
@pytorchbot merge -f 'Landed internally' (Initiating merge automatically since Phabricator Diff has merged, using force because this PR might not pass merge_rules.json but landed internally) |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Summary:
_wrapped_fns_to_patch points to f_globals which might change during iteration due to factors like lazy imports. This diff fixes potential runtime errors like:
Test Plan: CI
Reviewed By: Kronuz
Differential Revision: D50283983