Davids048
becd379f58
Detail:
- Merged LTX2 sampling behavior into the global registry and removed the ambiguous local converted-path default.
- Replaced single LTX2 sampling mapping with explicit model-ID mappings:
- Lightricks/LTX-2 -> LTX2BaseSamplingParam
- FastVideo/LTX2-base -> LTX2BaseSamplingParam
- FastVideo/LTX2-Distilled-Diffusers -> LTX2DistilledSamplingParam
- Kept LTX2T2VConfig as the pipeline config for all explicitly mapped LTX2 IDs.
- Removed implicit mapping for converted/ltx2_diffusers to avoid guessing base vs distilled for user-local
conversions.
- Added focused local tests at tests/local_tests/test_ltx2_registry.py for:
- exact base/distilled sampling resolution,
- pipeline config resolution,
- no fallback behavior for ambiguous local converted paths.
Assumptions:
- Canonical rename/ID intent:
- “base” names map to LTX2BaseSamplingParam.
- “Distilled” names map to LTX2DistilledSamplingParam.
- converted/ltx2_diffusers is intentionally ambiguous across users and must not be auto-assigned.
- Unknown/non-canonical names containing “LTX”/“LTX2” (but not matching explicit registered IDs) should not auto-
resolve to base or distilled.
- Sampling resolver returns None (caller falls back to generic defaults/user overrides).
- Pipeline config lookup raises a “No match found” error.
Notes:
- This change prioritizes explicitness over convenience: only predetermined, canonical model IDs get LTX2-specific
defaults; everything else requires user intent.
Local LTX-2 Tests
These tests depend on a checked-out LTX-2/ directory under the repo root
(FastVideo/LTX-2). Without that local repo, the tests will skip or fail.
For the CI-backed test suite, see fastvideo/tests/. Those are the tests
currently exercised in CI. (Eventually all tests will move to tests/.)