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

Tidy neutral physics in input.nml #197

Closed
aekiss opened this issue May 6, 2020 · 5 comments
Closed

Tidy neutral physics in input.nml #197

aekiss opened this issue May 6, 2020 · 5 comments

Comments

@aekiss
Copy link
Contributor

aekiss commented May 6, 2020

Following https://arccss.slack.com/archives/C9Q7Y1400/p1588727336089300

MOM's neutral physics settings in &ocean_nphysics_util_nml in input.nml are confusing because many ignored parameters are specified.

The ACCESS-OM2 configurations are explained in section 3.2.5 of the technical report, and the relevant code is ocean_nphysics_util.F90.

Here are the changes I think should be made to clarify &ocean_nphysics_util_nml in input.nml:

for the 0.1 deg configurations:

  • remove everything in ocean_nphysics_util_nml - ignored because use_this_module = .false. in &ocean_nphysics_nml

for the 1 deg and 0.25 deg configurations:

  • remove agm - ignored because agm_closure = .true.
  • remove rossby_radius_min and rossby_radius_max - ignored because agm_closure_length_rossby = .false.
  • remove agm_closure_eden_gamma - ignored because agm_closure_eden_greatbatch = .false.
  • remove vel_micom - ignored because tracer_mix_micom = .false.
  • remove agm_damping_time - ignored because agm_smooth_time = .false.
  • remove these because they take the default value .false.
    • agm_closure_length_rossby
    • agm_closure_eden_greatbatch
    • tracer_mix_micom
    • agm_closure_length_fixed
    • drhodz_smooth_horz
    • drhodz_smooth_vert
    • agm_closure_length_bczone
    • agm_smooth_space
    • agm_smooth_time

Does this look OK @russfiedler, @rmholmes, @AndyHoggANU ?

@AndyHoggANU
Copy link
Contributor

Looks OK to me. Thanks.

aekiss added a commit to COSIMA/01deg_jra55_iaf that referenced this issue May 6, 2020
aekiss added a commit to COSIMA/01deg_jra55_ryf that referenced this issue May 6, 2020
aekiss added a commit to COSIMA/025deg_jra55_iaf that referenced this issue May 6, 2020
aekiss added a commit to COSIMA/025deg_jra55_ryf that referenced this issue May 6, 2020
aekiss added a commit to COSIMA/1deg_jra55_iaf that referenced this issue May 6, 2020
aekiss added a commit to COSIMA/1deg_jra55_ryf that referenced this issue May 6, 2020
@StephenGriffies
Copy link

@aekiss I agree with removing those nml options that are not used in the configuration. neutral physics has heaps of options and best to focus just on those that are used by the configuration.

@rmholmes
Copy link
Collaborator

rmholmes commented May 6, 2020

Good idea, thanks Andrew.

I imagine there is many similar non-neccessary default options in lots of the other namelists where a similar thing could be done.

@aekiss
Copy link
Contributor Author

aekiss commented May 7, 2020

Agreed, there is plenty more redundancy in the namelists. Checking them is a bit of a tedious slog though (e.g. some defaults are true rather than false, etc)...

@aekiss aekiss closed this as completed May 14, 2020
@aekiss
Copy link
Contributor Author

aekiss commented May 14, 2020

Closed - I've made these changes in the ak-dev branches in all JRA configs

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

4 participants