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

Add tidal phase info and 3 long term components (Mm,Mf and Ssa) #337

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

mabelcalim
Copy link

Explicit tidal forcing in OGCM (ocean general circulation models) using tides in MOM5 plus accurate phase information is introduced.

Adding tidal forcing to ext-mode from 8-lunar/solar constituents+ long term (Ssa,Mf and Mm) and phase information based on TPXO 7.2. Phase information hardwired in. TPXO7.2 from constit.h.

Some tips to use tides in MOM5

  • must use vertical_coordinate z* to enable tides in the surface
  • turn off Bryan Lewis vertical diffusity in ocean_vert_mix

@aidanheerdegen
Copy link
Contributor

Hi @mabelcalim, sorry for taking so long to reply to this.

Your code changes are on top of a very old version of MOM from 2015

https://github.com/mabelcalim/mom5-tides/commits/master

Can you rebase your changes on top of the most recent commits to the MOM5 code, then we can look at any substantive code issues.

@aidanheerdegen aidanheerdegen self-assigned this Jun 8, 2021
@aidanheerdegen
Copy link
Contributor

Hi @mabelcalim

Do you need assistance with rebasing your code?

Cheers

Aidan

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