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

[dynamo] Turn on guard_nn_modules #125202

Closed
wants to merge 25 commits into from
Closed
Changes from 24 commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
bd72ff3
[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules
anijain2305 Apr 29, 2024
9ebc9cc
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 Apr 29, 2024
01ca0dc
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 Apr 30, 2024
9de7bb3
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 Apr 30, 2024
81de74d
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 Apr 30, 2024
483958a
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 Apr 30, 2024
ef705d1
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 1, 2024
79918db
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 1, 2024
43aa574
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 1, 2024
1081917
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 1, 2024
3c9ebd7
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 1, 2024
6cebb2e
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 2, 2024
70cf8ed
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 2, 2024
ec9056b
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 2, 2024
74c4dab
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 2, 2024
f9bf861
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 2, 2024
6ce1374
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 2, 2024
fce55cb
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 3, 2024
881ef68
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 8, 2024
a44202f
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 8, 2024
6ad81f5
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 8, 2024
3fd6224
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 8, 2024
ebdb5c9
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 9, 2024
80003cf
Update on "[DONT MERGE][FOR CI][dynamo] Turn on guard_nn_modules"
anijain2305 May 9, 2024
bca250a
Update on "[dynamo] Turn on guard_nn_modules"
anijain2305 May 10, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion torch/_dynamo/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@
allow_ignore_mark_dynamic = False

# Set this to False to assume nn.Modules() contents are immutable (similar assumption as freezing)
guard_nn_modules = False
guard_nn_modules = True

# Uses CPython internal dictionary tags to detect mutation. There is some
# overlap between guard_nn_modules_using_dict_tags and guard_nn_modules flag.
Expand Down