Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Monthly output instead of daily / nstep #33

Open
mnlevy1981 opened this issue Jul 1, 2022 · 1 comment
Open

Monthly output instead of daily / nstep #33

mnlevy1981 opened this issue Jul 1, 2022 · 1 comment
Labels
critical Issues with this label are top priority for the next PRs

Comments

@mnlevy1981
Copy link
Collaborator

Currently FEISTY is just dumping output to netcdf at the end of every time step; monthly averages should be sufficient and will reduce output size by quite a bit.

@mnlevy1981
Copy link
Collaborator Author

Kristen was converting from daily to monthly in postprocessing with

ds_mon = ds_out.resample({'time':'M'}).mean(dim='time')

A first pass to this issue could be including a resample_output flag in the output portion of the configuration file and then updating ds_out after the call to config_and_run_from_dataset() but prior to writing the output (well, we'd want to write the restart file then resample then write the history file)

@mnlevy1981 mnlevy1981 added the critical Issues with this label are top priority for the next PRs label Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
critical Issues with this label are top priority for the next PRs
Projects
None yet
Development

No branches or pull requests

1 participant