Skip to content

Support PyTorch < 2.7 Dynamo recompile-limit names in DA3 inference - #40

Open
sarlinpe wants to merge 1 commit into
fix/viewer-symlink-previewsfrom
fix/da3-dynamo-limits
Open

sarlinpe wants to merge 1 commit into
fix/viewer-symlink-previewsfrom
fix/da3-dynamo-limits

Conversation

@sarlinpe

@sarlinpe sarlinpe commented Oct 9, 2026

Copy link
Copy Markdown
Member

Summary

PyTorch 2.7 renamed the Dynamo options cache_size_limit / fail_on_cache_limit_hit to recompile_limit / fail_on_recompile_limit_hit, so patching the new names fails on PyTorch 2.6. DA3 inference now resolves the names supported by the installed PyTorch once (_dynamo_recompile_limits) and sets the limit values in a single place.

Testing

Patched config verified on PyTorch 2.6 (cache_size_limit=64, fail_on_cache_limit_hit=True, restored on exit).

@sarlinpe
sarlinpe added this pull request to stack #43 October 9, 2026 15:45
PyTorch 2.7 renamed cache_size_limit / fail_on_cache_limit_hit to recompile_limit /
fail_on_recompile_limit_hit; patching the new names raises on older releases. Resolve the
names once and set the limit values in a single place.
@sarlinpe
sarlinpe force-pushed the fix/da3-dynamo-limits branch from 0258c72 to ab986b6 Compare October 10, 2026 00:07

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant