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

Adding the Gradient Model into MOM6 #1642

Closed
wants to merge 9 commits into from

Conversation

sinakhani
Copy link
Contributor

I would like to request a "pull request review" for a newly added lateral mixing (mesoscale eddy) parameterization into MOM6. Changes are includes in the following files:

  1. MOM_lateral_mixing_coeffs.F90
  2. MOM_thickness_diffuse.F90
  3. MOM.F90

I look forward to working with you on merging these changes. Thank you.

@marshallward
Copy link
Collaborator

@sinakhani Could you move this PR to one of the development nodes? For example:

Your profile says that you are in Boulder, so perhaps this is in collaboration with NCAR, but if this is independent work then it is probably best handled at NOAA-GFDL.

I am also seeing over 300 changed files, and the PR does not contain any explanation about what has been changed. At the least, I think you may need to update your code. I would also suggest you investigate why several of the tests have failed.

@sinakhani
Copy link
Contributor Author

@marshallward I have this PR on my 'main' branch (https://github.com/sinakhani/MOM6/tree/main). My changes include in three .F90 files (MOM_lateral_mixing_coeffs.F90, MOM_thickness_diffuse.F90, and MOM.F90).

Mainly, in MOM_lateral_mixing_coeffs.F90, I have calculated "gradient slopes U_xH_x, etc" as defined in lines 933-978. Then, I calculate the gradient eddy transport diagnostics on u- and v-points in this file. Later, the calculated gradient transports are read in MOM_thickness_diffuse.F90 file to update KH term on u- and v-points. I hope this explanation could help how the new parameterization works.

Also, I am not sure why those tests are failed. I think this is not related to the part of the code I have added. Can I have a meeting with you to make sure that I am doing the "pull request" right? Thank you for your consideration.

@marshallward
Copy link
Collaborator

marshallward commented Oct 23, 2024

@sinakhani What I was trying to say above is that main (at mom-ocean/MOM6) is the public release of MOM6. But all PR contributions must be submitted to the one of the development branches, such as dev/gfdl which is maintained at https://github.com/NOAA-GFDL/MOM6.

PRs from individual contributors are reviewed and merged through the development branch. The main branch only receives updates from these development branches, not individuals.

I am going to close this PR, but I just wanted to clarify that you should re-open it to another fork, most likely https://github.com/NOAA-GFDL/MOM6.

@marshallward
Copy link
Collaborator

Forgot to add, feel free to reach our to me or others at GFDL, we can talk more about what is needed.

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