Replaces deprecated --headless with --viz none in mimic tests

[njawale42]

Description

This PR replaces --headless with --viz none in the subprocess command built by each test.
Files updated:

• source/isaaclab_mimic/test/test_generate_dataset_franka_state.py (2 occurrences)
• source/isaaclab_mimic/test/test_generate_dataset_franka_visuomotor.py (1 occurrence)
• source/isaaclab_mimic/test/test_generate_dataset_gr1t2_nutpour.py (1 occurrence)
• source/isaaclab_mimic/test/test_generate_dataset_gr1t2_pickplace.py (1 occurrence)
• source/isaaclab_mimic/test/test_generate_dataset_skillgen.py (1 occurrence)

Total: 5 files, 6 subprocess commands updated.

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist

• I have read and understood the contribution guidelines
• I have run the pre-commit checks with ./isaaclab.sh --format
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• I have updated the changelog and the corresponding version in the extension's config/extension.toml file
• I have added my name to the CONTRIBUTORS.md or my name already exists there

[isaaclab-review-bot]

🤖 Isaac Lab Review Bot

Summary

This PR replaces the deprecated --headless CLI flag with --viz none across 5 mimic test files (6 occurrences total) to silence AppLauncher deprecation warnings. The change is straightforward and correct — both flags achieve headless execution, but --viz none is the current recommended approach.

Architecture Impact

Self-contained. The changes only affect test subprocess commands that invoke external scripts (annotate_demos.py, generate_dataset.py). No core framework code is modified. The invoked scripts already support --viz none as the non-deprecated alternative to --headless.

Implementation Verdict

Minor fixes needed — The code change is correct, but there's an RST formatting error in the changelog.

Test Coverage

This PR modifies test files themselves (fixing deprecation warnings in test infrastructure). The tests will self-validate — if the --viz none flag weren't properly supported by the target scripts, these tests would fail. No additional test coverage needed.

CI Status

No CI checks available yet. Once CI runs, the mimic tests (@pytest.mark.isaacsim_ci) should pass, which would validate the flag replacement works correctly.

Findings

🔴 Critical: source/isaaclab_mimic/docs/CHANGELOG.rst:7 — RST formatting error will break documentation build

The Changed section underline uses ^^^^^ (5 characters) but must match the length of the header text "Changed" (7 characters). RST requires the underline to be at least as long as the text above it:

Changed
^^^^^

Should be:

Changed
^^^^^^^

This will cause Sphinx documentation build failures.

🔵 Improvement: source/isaaclab_mimic/docs/CHANGELOG.rst:9 — Changelog entry uses future date

The version 1.2.5 (2026-04-23) has a date in 2026, which appears to be a typo (should likely be 2025). This is consistent with other entries in the file that also use 2026/2025 dates, so this may be intentional for the project, but worth verifying the date convention.

🔵 Improvement: Consistency observation across test files

The change is applied consistently across all 5 test files. The pattern "--viz", "none" (as two separate list elements) is correct for subprocess argument passing and matches how other flags are structured in these command lists.

[greptile-apps]

Greptile Summary

This PR replaces the deprecated --headless CLI flag with --viz none across all 6 subprocess commands in the isaaclab_mimic test suite, silencing the AppLauncher deprecation warning. The version is bumped to 1.2.5 and a CHANGELOG entry is added.

Confidence Score: 5/5

Safe to merge; the only finding is a minor RST underline formatting issue in the CHANGELOG.

All code changes are straightforward, consistent, and correct. The single finding (RST underline length) is a P2 documentation style issue that does not affect runtime behavior.

source/isaaclab_mimic/docs/CHANGELOG.rst — RST underline for "Changed" is one character short.

Important Files Changed

Filename	Overview
source/isaaclab_mimic/docs/CHANGELOG.rst	Adds 1.2.5 entry; the "Changed" section underline (^^^^^) is 5 chars but the title is 7, which is invalid RST.
source/isaaclab_mimic/config/extension.toml	Version bumped from 1.2.4 to 1.2.5, consistent with CHANGELOG entry.
source/isaaclab_mimic/test/test_generate_dataset_franka_state.py	Replaces --headless with --viz none in two subprocess command lists; change is correct and consistent.
source/isaaclab_mimic/test/test_generate_dataset_franka_visuomotor.py	Replaces --headless with --viz none; correct fix.
source/isaaclab_mimic/test/test_generate_dataset_gr1t2_nutpour.py	Replaces --headless with --viz none; correct fix.
source/isaaclab_mimic/test/test_generate_dataset_gr1t2_pickplace.py	Replaces --headless with --viz none; correct fix.
source/isaaclab_mimic/test/test_generate_dataset_skillgen.py	Replaces --headless with --viz none; correct fix.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Test file invokes subprocess] --> B{Build command list}
    B --> C["--viz"]
    C --> D["none"]
    D --> E[run_script / subprocess executes]
    E --> F[AppLauncher receives --viz none]
    F --> G[Headless mode, no deprecation warning]

    style C fill:#90EE90
    style D fill:#90EE90

Loading

_{Reviews (1): Last reviewed commit: "Bumps isaaclab_mimic to 1.2.5 with --hea..." | Re-trigger Greptile}

[greptile-apps]

RST underline too short for section header

In reStructuredText, the underline must be at least as long as the title text. "Changed" is 7 characters, but the underline ^^^^^ is only 5 characters — this will produce a Sphinx/docutils warning (or error depending on config) when the docs are built. All other "Changed" headers in this file use ^^^^^^^.

Suggested change

	^^^^^
	^^^^^^^