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

Remove redundant ClimaAtmos checkpoint #1218

Open
Sbozzolo opened this issue Mar 6, 2025 · 0 comments
Open

Remove redundant ClimaAtmos checkpoint #1218

Sbozzolo opened this issue Mar 6, 2025 · 0 comments

Comments

@Sbozzolo
Copy link
Member

Sbozzolo commented Mar 6, 2025

#1179 introduces checkpoints for ClimaCoupler. As implemented in the PR, ClimaAtmos produces two checkpoints at the time instead of one. The reason why the choice was made is that ClimaAtmos does not provide a simple way to start a simulation from a time that is not 0, except when restarting from its own checkpoint. So, in order to have aligned times between component models, the simplest thing to do was to pick up from the previous checkpoint. The better solution for this might be to simply allow for a different t_start in ClimaAtmos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant