Reinstate DynamicSmagorinsky tests

[glwagner]

Opening this PR so we can continue tinkering with the DynamicSmagorinsky tests to see if we can get them to run.

cc @tomchor

[tomchor]

@glwagner thanks for opening this. Let me know how you'd like me to help here.

[tomchor]

Just FYI, dynamic Smagorinskys compile and run successfully over at Oceanostics: tomchor/Oceanostics.jl#183

[glwagner]

Then perhaps the cause is the flag -O0 that we use for init.

You could try adding a github actions build here. Might be nice to have that anyways

[tomchor]

Then perhaps the cause is the flag -O0 that we use for init.

I tried that locally and it didn't work (at least on my laptop).

You could try adding a github actions build here. Might be nice to have that anyways

Just for the turbulence tests? What are you envisioning?

[glwagner]

Then perhaps the cause is the flag -O0 that we use for init.

I tried that locally and it didn't work (at least on my laptop).

You could try adding a github actions build here. Might be nice to have that anyways

Just for the turbulence tests? What are you envisioning?

Is there a need to add more tests to it? I mean in the context of this PR, to get DynamicSmagorinsky tests to pass.

[tomchor]

You could try adding a github actions build here. Might be nice to have that anyways

Just for the turbulence tests? What are you envisioning?

Is there a need to add more tests to it? I mean in the context of this PR, to get DynamicSmagorinsky tests to pass.

I don't think we need to add more tests for dynamic Smagorinsky if that's your question. The tests we have should be fine, we just need to make them pass.

[glwagner]

You could try adding a github actions build here. Might be nice to have that anyways

Just for the turbulence tests? What are you envisioning?

Is there a need to add more tests to it? I mean in the context of this PR, to get DynamicSmagorinsky tests to pass.

I don't think we need to add more tests for dynamic Smagorinsky if that's your question. The tests we have should be fine, we just need to make them pass.

No, I'm not saying we should add more tests. I'm suggesting we move the turbulence closure tests to github actions in order to get them to pass.

[tomchor]

You could try adding a github actions build here. Might be nice to have that anyways

Just for the turbulence tests? What are you envisioning?

Is there a need to add more tests to it? I mean in the context of this PR, to get DynamicSmagorinsky tests to pass.

I don't think we need to add more tests for dynamic Smagorinsky if that's your question. The tests we have should be fine, we just need to make them pass.

No, I'm not saying we should add more tests. I'm suggesting we move the turbulence closure tests to github actions in order to get them to pass.

Yeah, if there isn't much onus to with having another test suite to deal with, I'm okay with that. I think we can only test CPU architecture there though.

[glwagner]

This suggests GPUs are available too https://github.blog/changelog/2024-07-08-github-actions-gpu-hosted-runners-are-now-generally-available/

[tomchor]

It seems you need an enterprise account to use them:

Does Clima have such an account?

[glwagner]

It seems you need an enterprise account to use them:

Does Clima have such an account?

I don't think so. So yeah, we would be limited to CPU tests. It could be better than nothing?