Isaac Lab 3.0 is GA. One missing close() locked log files on Windows

Description and use

In Isaac Lab 3.0, two tests fail on Windows due to open log files. A one‑line change that closes removed handlers fixes the issue, making all tests pass.

Problem

Two unit tests in Isaac Lab 3.0 failed on Windows because log files remained open after reconfiguration, causing PermissionError: [WinError 32].

Outcome

All logger tests now pass on Windows; the fix aligns Isaac Lab’s logging with Python’s logging.basicConfig behavior and prevents file lock regressions on any OS.

Workflow

• 1. Identify failing tests with PermissionError: [WinError 32].
• 2. Inspect configure_logging to see it removes but does not close old FileHandlers.
• 3. Add a loop to close each removed handler: for handler in root_logger.handlers[:]: root_logger.removeHandler(handler); handler.close()
• 4. Update tests to reconfigure logging before cleanup and assert handler stream is closed.
• 5. Submit PR #8809, pass CI, merge within 28 hours.

Evidence

When Isaac Lab reconfigured its logging, it unhooked the old handlers from the logger but never closed them, so the old log file stayed open in the background.

Source

Close what you remove. The loop that unhooks each old handler now also closes it: for handler in root_logger.handlers[:]: root_logger.removeHandler(handler) handler.close()

Source

Both now reconfigure logging before cleanup, which releases the file, and one of them asserts that the replaced handler's stream is gone.

Source