From cb064474262011108f91bc1a5863d1a68074ec3f Mon Sep 17 00:00:00 2001 From: CalamitousFelicitousness Date: Wed, 26 Aug 2026 03:58:58 +0100 Subject: [PATCH] fix(video): report a cancelled generation as 499 process_images swallows the interrupt assertion, so a cancel reaches the shared video core as an empty result and was raised as 'processing failed' with 500, leaving clients unable to tell a cancel from a crash. Mirrors the LTX path, which already returns 499. --- modules/video_models/video_run.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/modules/video_models/video_run.py b/modules/video_models/video_run.py index 97f76150c..db9ee06b8 100644 --- a/modules/video_models/video_run.py +++ b/modules/video_models/video_run.py @@ -307,6 +307,10 @@ def run(selected: models_def.Model, *, if err: raise VideoError(err, 500) if processed is None or (len(processed.images) == 0 and processed.bytes is None): + # process_images swallows the interrupt assertion, so an empty result is the only place + # a cancel and a genuine failure are still distinguishable + if shared.state.interrupted or shared.state.skipped: + raise VideoError('interrupted', 499) raise VideoError('processing failed', 500) log.info(f'Video: name="{selected.name}" cls={shared.sd_model.__class__.__name__} frames={len(processed.images)} time={t1-t0:.2f}')