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

Convergence tests pipeline takes several hours to finish #1420

Closed
ali-ramadhan opened this issue Mar 4, 2021 · 2 comments
Closed

Convergence tests pipeline takes several hours to finish #1420

ali-ramadhan opened this issue Mar 4, 2021 · 2 comments
Labels
testing 🧪 Tests get priority in case of emergency evacuation

Comments

@ali-ramadhan
Copy link
Member

Most of the convergence tests run in a reasonable amount of time except for a few:

For example: https://buildkite.com/clima/oceananigans-validation-experiments/builds/137

  • CPU Gaussian advection-diffusion: ~2:40 hours
  • GPU Gaussian advection-diffusion: ~1:45 hours
  • CPU forced flow free-slip: ~8 hours

I think for the Gaussian advection-diffusion tests we can just reduce the number of iterations. Some tests run for 26214 iterations.

Not sure why the CPU forced flow free-slip is so slow though. It's a 2D simulation. The CPU forced flow fixed-slip is similar I think and takes only ~20 minutes.

@ali-ramadhan ali-ramadhan added the testing 🧪 Tests get priority in case of emergency evacuation label Mar 4, 2021
@glwagner
Copy link
Member

glwagner commented Mar 4, 2021

Uhh weird. I guess we have to proof read those scripts. The free-slip and fixed-slip experiments just differ by boundary conditions I think.

@glwagner
Copy link
Member

I'm closing this issue because I'm judging that it's not of current, timely relevance to Oceananigans development. If you would like to make it a higher priority or if you think the issue was closed in error please feel free to re-open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing 🧪 Tests get priority in case of emergency evacuation
Projects
None yet
Development

No branches or pull requests

2 participants