Add explicit output format override

[houjun]

Summary

• add an explicit --output-format override with validated point, opensees, and essi modes
• keep the current defaults when no override is provided, but reject unsupported input/output combinations early
• document the new behavior and add regression coverage for parser, helper, and real-data conversion paths

Testing

• /global/cfs/cdirs/m3354/perl/conda_eqsim/bin/python -m unittest discover -s tests -p 'test_*.py'
• python3 -m py_compile convert.py tests/test_convert_cli.py tests/test_convert_helpers.py tests/test_convert_fixture_data.py

[Copilot]

Pull request overview

This PR adds an explicit --output-format override to the conversion CLI, while preserving existing defaults per input type and rejecting unsupported input/output combinations early.

Changes:

• Introduces --output-format {point,opensees,essi} and centralizes validation/defaulting via resolve_output_mode().
• Refactors output file naming/copy behavior via get_output_filename() and prepare_essi_output_file().
• Updates documentation and expands regression tests covering CLI parsing, helper behavior, and real-data conversion paths.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
convert.py	Adds output-format override support, validation, output naming, and ESSI template preparation logic.
README.md	Documents default output behavior and constraints for --output-format.
tests/test_convert_cli.py	Adds CLI parser coverage for the new --output-format flag.
tests/test_convert_helpers.py	Adds unit tests for output-mode resolution and output filename behavior.
tests/test_convert_fixture_data.py	Adds integration/regression tests for explicit output mode support and rejection of invalid combos.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[houjun]

@copilot apply changes based on the comments in this thread

[Copilot]

@copilot apply changes based on the comments in this thread

Implemented in commit 54548af. I applied the review-thread MPI fixes in both paths: ESSI file copy now runs only on rank 0, followed by an MPI barrier before generate_acc_dis_time(), and validated with full unit tests plus parallel validation.