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 ocean relativeVorticityAtSurface to highFrequencyStats #6566

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mark-petersen
Copy link
Contributor

It is useful to visualize relative vorticity at the surface, which was not saved in high frequency stats in the past. I have added this commit to several recent simulations, including for the high-resolution Gulf of Mexico work. It works as expected. The variable is added to the default output for highFrequencyStats, but is only a 2D field and is optional.

We commonly use relative vorticity at the surface for images and movies, in addition to surface kinetic energy.

Copy link
Contributor

@xylar xylar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mark-petersen, I'm approving this based on a visual inspection of the code, which looks correct and is quite simple. Let me know if you would like me to do further testing.

@mark-petersen mark-petersen added the non-BFB PR makes roundoff changes to answers. label Aug 27, 2024
@mark-petersen
Copy link
Contributor Author

I labelled this as non-bfb because it changes the high frequency output variables. It is not answer changing.

@rljacob rljacob added this to the v3.1beta milestone Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mpas-ocean non-BFB PR makes roundoff changes to answers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants