fix(app): dedent on_tag_image to module level (was accidentally nested in _build_tagger_html, causing NameError)

app.py

@@ -537,105 +537,105 @@ def _build_tagger_html(result: dict, lc: str) -> str:
     return "\n".join(lines)
 
 
-    def on_tag_image(image, gen_threshold, char_threshold, fmt, lang, progress=gr.Progress(), skip_pose=False, with_caption=False, caption_kind="florence", control_kind="skeleton_black", pose_mode="yolo"):
-        lc = "ru" if lang == "RU" else "en"
-        # fmt carries "prompt|<mode>" from the tagger-mode dropdown: "prompt|wd:eva02", "prompt|ensemble".
-        if "|" in fmt:
-            fmt, mode = fmt.split("|", 1)
-            fmt = fmt.strip() or "prompt"
-            mode = mode.strip() or "ensemble"
-        else:
-            mode = "ensemble"
-        empty_chips = gr.update(choices=[], value=[], visible=False)
-        hidden_control = (gr.update(visible=False), gr.update(visible=False))
-        if image is None:
+def on_tag_image(image, gen_threshold, char_threshold, fmt, lang, progress=gr.Progress(), skip_pose=False, with_caption=False, caption_kind="florence", control_kind="skeleton_black", pose_mode="yolo"):
+    lc = "ru" if lang == "RU" else "en"
+    # fmt carries "prompt|<mode>" from the tagger-mode dropdown: "prompt|wd:eva02", "prompt|ensemble".
+    if "|" in fmt:
+        fmt, mode = fmt.split("|", 1)
+        fmt = fmt.strip() or "prompt"
+        mode = mode.strip() or "ensemble"
+    else:
+        mode = "ensemble"
+    empty_chips = gr.update(choices=[], value=[], visible=False)
+    hidden_control = (gr.update(visible=False), gr.update(visible=False))
+    if image is None:
+        return (
+            gr.update(value=f'<div style="color:#F87171;font-size:12px;">{t("tagger_no_image", lc)}</div>'),
+            gr.update(visible=False),
+            empty_chips, empty_chips,
+            gr.update(visible=False),
+            gr.update(visible=False),
+            "", "",
+        ) + hidden_control
+    try:
+        progress(0, desc=t("tagger_processing", lc))
+        if not _TAGGER_DEPS_OK:
+            msg = f'<div style="color:#FBBF24;font-size:13px;line-height:1.5;">{t("tagger_unavailable", lc)}</div>'
             return (
-                gr.update(value=f'<div style="color:#F87171;font-size:12px;">{t("tagger_no_image", lc)}</div>'),
+                gr.update(value=msg),
                 gr.update(visible=False),
                 empty_chips, empty_chips,
                 gr.update(visible=False),
                 gr.update(visible=False),
                 "", "",
             ) + hidden_control
-        try:
-            progress(0, desc=t("tagger_processing", lc))
-            if not _TAGGER_DEPS_OK:
-                msg = f'<div style="color:#FBBF24;font-size:13px;line-height:1.5;">{t("tagger_unavailable", lc)}</div>'
-                return (
-                    gr.update(value=msg),
-                    gr.update(visible=False),
-                    empty_chips, empty_chips,
-                    gr.update(visible=False),
-                    gr.update(visible=False),
-                    "", "",
-                ) + hidden_control
-            progress(0.35, desc=t("tagger_ratings_label", lc))
-            result = get_ensemble_tagger().tag_image(
-                image, gen_threshold, char_threshold,
-                mode=mode, skip_pose=skip_pose, with_caption=with_caption,
-                caption_kind=caption_kind, pose_mode=pose_mode,
-            )
-            progress(0.9, desc=t("tagger_results", lc))
-            if not result["general"] and not result["characters"] and not result.get("nl_caption"):
-                html_out = f'<div style="color:#FBBF24;font-size:12px;">{t("tagger_no_tags", lc)}</div>'
-                return (
-                    gr.update(value=html_out),
-                    gr.update(visible=False),
-                    empty_chips, empty_chips,
-                    gr.update(visible=False),
-                    gr.update(visible=False),
-                    "", "",
-                ) + hidden_control
-            html_out = _build_tagger_html(result, lc)
-            general_names = list(result["general"].keys())[:60]
-            char_names = list(result["characters"].keys())
-
-            # Merge the NL caption into the tag chips so it can be picked up by
-            # the same apply/copy pipeline as booru tags. Pose tags are already
-            # folded into `general`/`caption`/`taglist` by the ensemble tagger,
-            # so they must NOT be added here again.
-            extra_names: list[str] = []
-            if result.get("nl_caption"):
-                extra_names.append(result["nl_caption"].strip())
-
-            all_names = extra_names + general_names
-            general_choices = [(n, n) for n in all_names]
-            char_choices = [(n.replace("_", " "), n) for n in char_names]
-
-            raw_caption = result["caption"]
-            esc_taglist = result["taglist"]
-            tags_box = esc_taglist if fmt == "prompt" else raw_caption
-
-            # ControlNet-style preview built from the same keypoints the pose tags
-            # came from (pure numpy / PIL, no extra models). Hidden when the pose
-            # pass was skipped or the preview kind has nothing to show.
-            control_outputs = _render_control_outputs(
-                ImageOps.exif_transpose(Image.fromarray(image)),
-                result, control_kind,
-            )
-
+        progress(0.35, desc=t("tagger_ratings_label", lc))
+        result = get_ensemble_tagger().tag_image(
+            image, gen_threshold, char_threshold,
+            mode=mode, skip_pose=skip_pose, with_caption=with_caption,
+            caption_kind=caption_kind, pose_mode=pose_mode,
+        )
+        progress(0.9, desc=t("tagger_results", lc))
+        if not result["general"] and not result["characters"] and not result.get("nl_caption"):
+            html_out = f'<div style="color:#FBBF24;font-size:12px;">{t("tagger_no_tags", lc)}</div>'
             return (
                 gr.update(value=html_out),
-                gr.update(visible=True),
-                gr.update(choices=general_choices, value=all_names, visible=True),
-                gr.update(choices=char_choices, value=char_names, visible=True),
-                gr.update(value=tags_box, visible=True),
-                gr.update(visible=True),
-                raw_caption,
-                esc_taglist,
-            ) + control_outputs
-        except Exception as exc:
-            import traceback
-            traceback.print_exc()
-            err = t("tagger_error", lc).format(exc=html.escape(str(exc)))
-            return (
-                gr.update(value=f'<div style="color:#F87171;font-size:12px;">{err}</div>'),
                 gr.update(visible=False),
                 empty_chips, empty_chips,
                 gr.update(visible=False),
                 gr.update(visible=False),
                 "", "",
             ) + hidden_control
+        html_out = _build_tagger_html(result, lc)
+        general_names = list(result["general"].keys())[:60]
+        char_names = list(result["characters"].keys())
+
+        # Merge the NL caption into the tag chips so it can be picked up by
+        # the same apply/copy pipeline as booru tags. Pose tags are already
+        # folded into `general`/`caption`/`taglist` by the ensemble tagger,
+        # so they must NOT be added here again.
+        extra_names: list[str] = []
+        if result.get("nl_caption"):
+            extra_names.append(result["nl_caption"].strip())
+
+        all_names = extra_names + general_names
+        general_choices = [(n, n) for n in all_names]
+        char_choices = [(n.replace("_", " "), n) for n in char_names]
+
+        raw_caption = result["caption"]
+        esc_taglist = result["taglist"]
+        tags_box = esc_taglist if fmt == "prompt" else raw_caption
+
+        # ControlNet-style preview built from the same keypoints the pose tags
+        # came from (pure numpy / PIL, no extra models). Hidden when the pose
+        # pass was skipped or the preview kind has nothing to show.
+        control_outputs = _render_control_outputs(
+            ImageOps.exif_transpose(Image.fromarray(image)),
+            result, control_kind,
+        )
+
+        return (
+            gr.update(value=html_out),
+            gr.update(visible=True),
+            gr.update(choices=general_choices, value=all_names, visible=True),
+            gr.update(choices=char_choices, value=char_names, visible=True),
+            gr.update(value=tags_box, visible=True),
+            gr.update(visible=True),
+            raw_caption,
+            esc_taglist,
+        ) + control_outputs
+    except Exception as exc:
+        import traceback
+        traceback.print_exc()
+        err = t("tagger_error", lc).format(exc=html.escape(str(exc)))
+        return (
+            gr.update(value=f'<div style="color:#F87171;font-size:12px;">{err}</div>'),
+            gr.update(visible=False),
+            empty_chips, empty_chips,
+            gr.update(visible=False),
+            gr.update(visible=False),
+            "", "",
+        ) + hidden_control
 
 
 def _render_control_outputs(orig_pil, result, control_kind):