Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 8b81b26

Browse filesBrowse files
williamwen42williamwen42
committed
Update on "[dynamo, nested graph breaks] refactor codegen to minimize NULL codegen'ing"
 Stop codegening NULLs that we need to pop later. Some output_graph.py changes to prepare for nested graph break support. cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames [ghstack-poisoned]
2 parents 35f9f70 + b2d8b99 commit 8b81b26
Copy full SHA for 8b81b26

File tree

Expand file treeCollapse file tree

1 file changed

+1
-0
lines changed
Filter options
Expand file treeCollapse file tree

1 file changed

+1
-0
lines changed

‎test/dynamo/test_error_messages.py

Copy file name to clipboardExpand all lines: test/dynamo/test_error_messages.py
+1Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -239,6 +239,7 @@ def fn(obj):
239239
Unsupported context manager
240240
Explanation: Dynamo does not know how to enter a `int` context manager.
241241
Hint: Avoid using the unsupported context manager.
242+
Hint: If the context manager seems like it should be supported (e.g. torch.set_grad_enabled), then it may be the case that it was created outside the compiled region, which Dynamo does not support. Supported context managers can cross graph break boundaries only if they are local non-closure variables, or are intermediate values.
242243
Hint: File an issue to PyTorch. Simple context managers can potentially be supported, but note that context managers can't be supported in general
243244
244245
Developer debug context: Attempted SETUP_WITH/BEFORE_WITH on ConstantVariable(int: 3)

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.