-
Notifications
You must be signed in to change notification settings - Fork 232
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
Conversation
@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. |
@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. |
@sinakhani What I was trying to say above is that PRs from individual contributors are reviewed and merged through the development branch. The 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. |
Forgot to add, feel free to reach our to me or others at GFDL, we can talk more about what is needed. |
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:
I look forward to working with you on merging these changes. Thank you.