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

Passing the runoff iron and geothermal heat fluxes to ocean biogeochem #677

Conversation

nikizadehgfdl
Copy link
Contributor

  • COBALT is updated to get the iron flux in frozen runoff (proxy to bergs iron)
    and the geothermal heat flux.

- COBALT is updated to get the iron flux in frozen runoff (proxy to bergs iron)
  and the geothermal heat flux.
- Optional argumnents to subroutine calls should be explicitly named
- This branch is compatible with gitlab branch user/nnz/cas_geotherm_cobalt_updates
  https://gitlab.gfdl.noaa.gov/fms/ocean_shared/tree/user/nnz/cas_geotherm_cobalt_updates
@nikizadehgfdl
Copy link
Contributor Author

This update is compatible with gitlab branch:

https://gitlab.gfdl.noaa.gov/fms/ocean_shared/tree/user/nnz/cas_geotherm_cobalt_updates

@adcroft
Copy link
Collaborator

adcroft commented Jan 17, 2018

@adcroft adcroft merged commit 26abd28 into mom-ocean:dev/gfdl Jan 18, 2018
marshallward pushed a commit to Hallberg-NOAA/MOM6 that referenced this pull request Sep 10, 2024
* (*)bugfixes for internal tides

- TKE_itidal_input values are set to zero where it is not applied to
  the model so that diagnostics are consistent

- removed useless halo updates

- diagnose energy loss terms as dE/dt instead of equivalent loss rate
  to properly close energy budget

- tot_vel_btTide2 was already a velocity squared, no need to square again

- Froude loss term was not properly initialized

- add missing halo updates for internal tide speed

- compute residual/critical slopes loss only where it has a meaning,
  allows to clean up noise in field

- uh/vh in energy flux routines do not need to be inout

* Refactor debugging internal tides

- put test for negative energt behind debug flag

- do energy budget in debug mode

- add flags to skip refraction, propagation for debugging

- add option to input TKE only at first step for debugging

- add checksums for unit scaling testing

- rewrite terms in refraction to avoid substractions of velocities

- rewrite gradient of coriolis in rotationally invariant form

* Add internal tides diffusivities

- MOM_internal_tides gets new routine that converts
  TKE losses into diffusivities using the TKE_to_Kd array

- MOM_set_diffusivities handles the diagnostics

* extend find_rho_bottom to get bbl thickness/index

* new calculation of BBL, profiles in H units
gmacgilchrist pushed a commit to gmacgilchrist/MOM6 that referenced this pull request Oct 9, 2024
* (*)bugfixes for internal tides

- TKE_itidal_input values are set to zero where it is not applied to
  the model so that diagnostics are consistent

- removed useless halo updates

- diagnose energy loss terms as dE/dt instead of equivalent loss rate
  to properly close energy budget

- tot_vel_btTide2 was already a velocity squared, no need to square again

- Froude loss term was not properly initialized

- add missing halo updates for internal tide speed

- compute residual/critical slopes loss only where it has a meaning,
  allows to clean up noise in field

- uh/vh in energy flux routines do not need to be inout

* Refactor debugging internal tides

- put test for negative energt behind debug flag

- do energy budget in debug mode

- add flags to skip refraction, propagation for debugging

- add option to input TKE only at first step for debugging

- add checksums for unit scaling testing

- rewrite terms in refraction to avoid substractions of velocities

- rewrite gradient of coriolis in rotationally invariant form

* Add internal tides diffusivities

- MOM_internal_tides gets new routine that converts
  TKE losses into diffusivities using the TKE_to_Kd array

- MOM_set_diffusivities handles the diagnostics

* extend find_rho_bottom to get bbl thickness/index

* new calculation of BBL, profiles in H units
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

Successfully merging this pull request may close these issues.

2 participants