LTX: drop the 'tested' log notice, a working model needs none
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Tw9fjYZhHqk196LykrWy3T
This commit is contained in:
@@ -83,16 +83,10 @@ def frames_for(length: int) -> int:
|
||||
return ((length - 1 + VIDEO_TEMPORAL_RATIO - 1) // VIDEO_TEMPORAL_RATIO) * VIDEO_TEMPORAL_RATIO + 1
|
||||
|
||||
|
||||
NOTE = ('LTX 2.3 is tested on a real model (Video to Video direct, Image to Video from References, separate loaders, LoRAs, Noise merging). '
|
||||
'Other enhancements are not tested on LTX yet. Problems: '
|
||||
'https://github.com/digital-garbage/FunUI/issues')
|
||||
|
||||
|
||||
def empty_latent(model, width, height, length, batch_size=1, audio_vae=None, frame_rate=25.0):
|
||||
"""The joint latent, or None when this is not an LTX audio+video model."""
|
||||
if not is_ltx(model):
|
||||
return None
|
||||
log.once("ltx_note", log.INFO, "FunUI LTX", NOTE)
|
||||
if audio_vae is None:
|
||||
raise RuntimeError(
|
||||
"this model makes sound with the picture and sizes it from the audio VAE: wire the audio "
|
||||
|
||||
@@ -188,14 +188,13 @@ def test_a_size_that_is_not_a_multiple_of_32_is_said(comfyui, monkeypatch):
|
||||
monkeypatch.setattr(log, "once", lambda key, level, source, msg: said.append(msg))
|
||||
m.empty_latent(object(), 816, 512, 9, 1, audio_vae=_AudioVae(), frame_rate=25.0)
|
||||
assert any("800x512" in m_ for m_ in said)
|
||||
assert any("tested on a real model" in m_ for m_ in said) # every LTX run says what is tested
|
||||
assert not any("tested" in m_ for m_ in said) # a working model needs no notice
|
||||
|
||||
|
||||
def test_ltx_presets_are_the_two_that_were_tested(registered):
|
||||
from core import routes
|
||||
titles = [p["title"] for _s, make in routes.modules().providers("pipeline_presets") for p in make() if p["id"].startswith("ltx")]
|
||||
assert len(titles) == 2 and not any("UNTESTED" in t for t in titles)
|
||||
assert "Noise merging" in _ltx().NOTE and "issues" in _ltx().NOTE
|
||||
|
||||
|
||||
@pytest.mark.parametrize("pid", ["ltx23_image_to_video_references", "ltx23_video_to_video_direct"])
|
||||
|
||||
Reference in New Issue
Block a user