Skip to content

Add --output-file and --output-frequency options - #211

Open
BenjaminRodenberg wants to merge 1 commit into
mainfrom
output-options
Open

BenjaminRodenberg wants to merge 1 commit into
mainfrom
output-options

Conversation

@BenjaminRodenberg

Copy link
Copy Markdown
Collaborator

The output file was always Output/model_output.nc and a record was written every 8 time steps, which is daily only at the default PT3H. --output-file sets the path. --output-frequency takes an ISO 8601 duration (default P1D) that must be a multiple of --time-step, so the output interval no longer changes with the time step.

Assisted-by: Claude Code:claude-opus-5-5

Author's Todos

  • I ran the pre-commit hooks and fixed all issues
  • I added an entry under develop in the CHANGELOG.md (may be skipped for minor changes not observable for the user)

The output file was always Output/model_output.nc and a record was
written every 8 time steps, which is daily only at the default PT3H.
--output-file sets the path. --output-frequency takes an ISO 8601
duration (default P1D) that must be a multiple of --time-step, so the
output interval no longer changes with the time step.

Assisted-by: Claude Code:claude-opus-5-5
@BenjaminRodenberg BenjaminRodenberg self-assigned this Oct 27, 2026
@BenjaminRodenberg BenjaminRodenberg added enhancement New feature or request usability Improves the user experience labels Oct 27, 2026
@BenjaminRodenberg BenjaminRodenberg added this to the v0.9.0 milestone Oct 27, 2026
@BenjaminRodenberg

Copy link
Copy Markdown
Collaborator Author

@hauke-schmidt these are the options you asked for. Hope I got this right.

This PR does not include the more flexible configuration of output fields (will go elsewhere). The MPI parallelization is not included here because #197 is still unmerged. Hope this is still sufficient for you to evaluate if it meets your requirements.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request usability Improves the user experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant