viz: limit rendering/slice CLI opts to --png/--mp4 only

TUI and interactive modes have their own UI controls for colormap,
vmin/vmax, log scale, and slice position, so those CLI flags are
only needed for batch rendering.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: sbryngelson

toolchain/mfc/cli/commands.py

@@ -935,7 +935,7 @@
         ),
         Argument(
             name="cmap",
-            help="Matplotlib colormap name (--interactive, --png, --mp4).",
+            help="Matplotlib colormap name (--png, --mp4 only).",
             type=str,
             default='viridis',
             metavar="CMAP",
@@ -967,14 +967,14 @@
         ),
         Argument(
             name="vmin",
-            help="Minimum value for color scale (--interactive, --png, --mp4).",
+            help="Minimum value for color scale (--png, --mp4 only).",
             type=float,
             default=None,
             metavar="VMIN",
         ),
         Argument(
             name="vmax",
-            help="Maximum value for color scale (--interactive, --png, --mp4).",
+            help="Maximum value for color scale (--png, --mp4 only).",
             type=float,
             default=None,
             metavar="VMAX",
@@ -988,7 +988,7 @@
         ),
         Argument(
             name="slice-axis",
-            help="Axis for 3D slice (--interactive, --png, --mp4).",
+            help="Axis for 3D slice (--png, --mp4 only).",
             type=str,
             default='z',
             choices=["x", "y", "z"],
@@ -997,15 +997,15 @@
         ),
         Argument(
             name="slice-value",
-            help="Coordinate value at which to take the 3D slice (--interactive, --png, --mp4).",
+            help="Coordinate value at which to take the 3D slice (--png, --mp4 only).",
             type=float,
             default=None,
             dest="slice_value",
             metavar="VAL",
         ),
         Argument(
             name="slice-index",
-            help="Array index at which to take the 3D slice (--interactive, --png, --mp4).",
+            help="Array index at which to take the 3D slice (--png, --mp4 only).",
             type=int,
             default=None,
             dest="slice_index",
@@ -1040,7 +1040,7 @@
         ),
         Argument(
             name="log-scale",
-            help="Logarithmic color/y scale (--interactive, --png, --mp4).",
+            help="Logarithmic color/y scale (--png, --mp4 only).",
             action=ArgAction.STORE_TRUE,
             default=False,
             dest="log_scale",
@@ -1102,20 +1102,21 @@
         ("--interactive / -i", "Dash web UI (1D/2D/3D, needs browser or SSH tunnel)"),
         ("--png", "Save PNG image(s) to case_dir/viz/"),
         ("--mp4", "Encode frames into an MP4 video"),
-        ("-- Appearance (--interactive, --png, --mp4) --", ""),
+        ("-- Rendering (--png, --mp4 only) --", ""),
         ("--cmap NAME", "Matplotlib colormap (default: viridis)"),
         ("--vmin / --vmax", "Fix color-scale limits"),
         ("--log-scale", "Logarithmic color/y axis"),
-        ("--dpi N", "Image resolution for --png/--mp4 (default: 150)"),
-        ("-- 3D slicing (--interactive, --png, --mp4) --", ""),
+        ("--dpi N", "Image resolution (default: 150)"),
+        ("-o, --output DIR", "Output directory (default: case_dir/viz/)"),
+        ("-- 3D slicing (--png, --mp4 only) --", ""),
         ("--slice-axis x|y|z", "Plane to slice (default: z midplane)"),
         ("--slice-value VAL", "Slice at coordinate value"),
         ("--slice-index IDX", "Slice at array index"),
-        ("-- Mode-specific --", ""),
-        ("-o, --output DIR", "Output directory for --png/--mp4 (default: case_dir/viz/)"),
-        ("--fps N", "Frames per second for --mp4 (default: 10)"),
-        ("--port PORT", "Web server port for --interactive (default: 8050)"),
-        ("--host HOST", "Bind address for --interactive (default: 127.0.0.1)"),
+        ("-- --mp4 only --", ""),
+        ("--fps N", "Frames per second (default: 10)"),
+        ("-- --interactive only --", ""),
+        ("--port PORT", "Web server port (default: 8050)"),
+        ("--host HOST", "Bind address (default: 127.0.0.1)"),
     ],
 )
 

toolchain/mfc/viz/viz.py

@@ -287,25 +287,6 @@ def viz():  # pylint: disable=too-many-locals,too-many-statements,too-many-branc
             f"No matching timesteps for --step {step_arg!r}{detail}. "
             f"Available steps: {_steps_hint(steps)}")
 
-    # Collect rendering options
-    render_opts = {
-        'cmap':  ARG('cmap'),
-        'dpi':   ARG('dpi'),
-        'slice_axis': ARG('slice_axis'),
-    }
-    if ARG('vmin') is not None:
-        render_opts['vmin'] = float(ARG('vmin'))
-    if ARG('vmax') is not None:
-        render_opts['vmax'] = float(ARG('vmax'))
-    if ARG('log_scale'):
-        render_opts['log_scale'] = True
-    if ARG('slice_index') is not None and ARG('slice_value') is not None:
-        raise MFCException("--slice-index and --slice-value are mutually exclusive.")
-    if ARG('slice_index') is not None:
-        render_opts['slice_index'] = int(ARG('slice_index'))
-    if ARG('slice_value') is not None:
-        render_opts['slice_value'] = float(ARG('slice_value'))
-
     interactive = ARG('interactive')
 
     # Lagrange bubble overlay: auto-detect D/lag_bubble_evol_*.dat files
@@ -364,11 +345,6 @@ def read_step(step):
             f"Use --list-vars to see variables at a given step."
         )
 
-    # Validate colormap early so all modes get a clean error for bad --cmap
-    cmap_name = ARG('cmap')
-    if cmap_name:
-        _validate_cmap(cmap_name)
-
     # TUI mode — launch Textual terminal UI (1D/2D only)
     if use_tui:
         if test_assembled.ndim == 3:
@@ -394,6 +370,29 @@ def read_step(step):
                         bubble_func=bubble_func)
         return
 
+    # --- PNG / MP4 rendering options (not used by TUI or interactive) ---
+    render_opts = {
+        'cmap':  ARG('cmap'),
+        'dpi':   ARG('dpi'),
+        'slice_axis': ARG('slice_axis'),
+    }
+    if ARG('vmin') is not None:
+        render_opts['vmin'] = float(ARG('vmin'))
+    if ARG('vmax') is not None:
+        render_opts['vmax'] = float(ARG('vmax'))
+    if ARG('log_scale'):
+        render_opts['log_scale'] = True
+    if ARG('slice_index') is not None and ARG('slice_value') is not None:
+        raise MFCException("--slice-index and --slice-value are mutually exclusive.")
+    if ARG('slice_index') is not None:
+        render_opts['slice_index'] = int(ARG('slice_index'))
+    if ARG('slice_value') is not None:
+        render_opts['slice_value'] = float(ARG('slice_value'))
+
+    cmap_name = ARG('cmap')
+    if cmap_name:
+        _validate_cmap(cmap_name)
+
     # Create output directory
     output_base = ARG('output')
     if output_base is None: