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

Adjust time step with radius #430

Merged
merged 3 commits into from
Jan 17, 2024
Merged

Adjust time step with radius #430

merged 3 commits into from
Jan 17, 2024

Conversation

milankl
Copy link
Member

@milankl milankl commented Jan 17, 2024

Previously the time step was automatically chosen as a function of resolution but not as a function of radius, but larger planets have effectively a coarser grid spacing, meaning they can deal with larger time steps. And vice versa: radius=1e6 currently blows up in the shallow water model because we effectively have a finer resolution by a factor of ~7x, which requires a ~7x smaller time step.

As suggested by @KristianJS

@milankl milankl added the time integration 🕙 Time integration of the equations label Jan 17, 2024
@milankl
Copy link
Member Author

milankl commented Jan 17, 2024

Galewsky jet after 4 days on Mars (radius=3.39e6)

image

Galewsky jet after 4 days on a planet 4x Earth's radius

image

However, at the moment larger planets cause stability problems, not sure why:

image

@milankl milankl merged commit c4818bd into main Jan 17, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
time integration 🕙 Time integration of the equations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant