Skip to content

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

Merged
Zador-Pataki merged 1 commit into
fix/viewer-symlink-previewsfrom
fix/da3-dynamo-limits
Oct 26, 2026
Merged

Zador-Pataki merged 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
@sarlinpe
sarlinpe force-pushed the fix/da3-dynamo-limits branch from 0258c72 to ab986b6 Compare October 10, 2026 00:07
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.
@Zador-Pataki
Zador-Pataki force-pushed the fix/da3-dynamo-limits branch from ab986b6 to 0307621 Compare October 26, 2026 09:48
@Zador-Pataki
Zador-Pataki merged commit 0c504d1 into main Oct 26, 2026
2 checks passed
@Zador-Pataki
Zador-Pataki deleted the fix/da3-dynamo-limits branch October 26, 2026 09:49
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.

2 participants