CARVIEW |
Navigation Menu
-
Notifications
You must be signed in to change notification settings - Fork 24.7k
fix test_lazy_module_kwargs #137705
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 test_lazy_module_kwargs #137705
Conversation
π Helpful Linksπ§ͺ See artifacts and rendered test results at hud.pytorch.org/pr/137705
Note: Links to docs will display an error until the docs builds have been completed. β You can merge normally! (3 Unrelated Failures)As of commit 80ac69e with merge base 93bbc8a ( FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
BROKEN TRUNK - The following job failed but were present on the merge base:π Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This pull request was exported from Phabricator. Differential Revision: D64185644 |
7ce7e9e
to
880e323
Compare
This pull request was exported from Phabricator. Differential Revision: D64185644 |
@@ -1527,6 +1527,14 @@ def _detect_attribute_assignment(mod: torch.nn.Module): | |||
"_state_dict_hooks", | |||
"_state_dict_pre_hooks", | |||
"training", | |||
] | |||
NN_MODULE_LAZY_STD_ATTRS = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are they previously special handled in dynamo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This only applies to non-strict, since this bug was in detecting arribute assigments (in general Dynamo errors so we don't run this detection there).
This pull request was exported from Phabricator. Differential Revision: D64185644 |
Summary: Pull Request resolved: pytorch#137705 Test Plan: fixed Differential Revision: D64185644
880e323
to
ab4c86a
Compare
Summary: Pull Request resolved: pytorch#137705 Test Plan: fixed Reviewed By: tugsbayasgalan Differential Revision: D64185644
ab4c86a
to
0ea1b80
Compare
This pull request was exported from Phabricator. Differential Revision: D64185644 |
Summary: Pull Request resolved: pytorch#137705 Test Plan: fixed Reviewed By: tugsbayasgalan Differential Revision: D64185644
This pull request was exported from Phabricator. Differential Revision: D64185644 |
0ea1b80
to
80ac69e
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 |
Test Plan: fixed
Differential Revision: D64185644