implement restart for internal tides

[raphaeldussin]

• add a call to register restart diabatic in MOM
• register restart diabatic call register restart internal tides
• internal tides restart uses the extra_axes optional argument
• support for extra_axes added to MOM_restart/MOM_io
• support for 4d array read with FMS1
• works with FMS2
• writes 4d restarts for horizontal, propagation angle and tidal frequency
• writes 5 restart variables for all the propagating vertical modes

[codecov]

Codecov Report

Merging #463 (11d24f3) into dev/gfdl (1577ae1) will decrease coverage by 0.03%.
The diff coverage is 16.56%.

❗ Current head 11d24f3 differs from pull request most recent head ca7d82b. Consider uploading reports for the commit ca7d82b to get more accurate results

@@             Coverage Diff              @@
##           dev/gfdl     #463      +/-   ##
============================================
- Coverage     38.04%   38.01%   -0.03%     
============================================
  Files           270      270              
  Lines         77637    77764     +127     
  Branches      14285    14320      +35     
============================================
+ Hits          29538    29564      +26     
- Misses        42783    42876      +93     
- Partials       5316     5324       +8     

Files Changed	Coverage Δ
...c/parameterizations/lateral/MOM_internal_tides.F90	0.00% <0.00%> (ø)
src/framework/MOM_restart.F90	32.28% <28.12%> (-0.49%)	⬇️
...parameterizations/vertical/MOM_diabatic_driver.F90	46.86% <44.44%> (+0.04%)	⬆️
src/framework/MOM_io.F90	29.75% <66.66%> (+0.74%)	⬆️
src/core/MOM.F90	50.89% <100.00%> (+0.12%)	⬆️

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

📢 Have feedback on the report? Share it here.

[marshallward]

Extending restart registration to set the global_file flag seems to me to be a very good solution to me for supporting potentially domain-decomposed 4d restarts.

It does revert some of the effort to transition control structures from pointers to stack variable, but this can be handled for now.

I have included some minor comments which should finalize this PR, and we can merge it ASAP.

[marshallward]

The updated version looks like a very good implementation to support 4d restart fields from the internal tide module, and allows for extended support in the future.