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

Test perf output #15

Closed
wants to merge 2 commits into from
Closed

Test perf output #15

wants to merge 2 commits into from

Conversation

marshallward
Copy link
Owner

testing perf output

Hallberg-NOAA and others added 2 commits February 29, 2024 09:28

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
  Revised the update_OBC_segment_data code to keep the z-space input data in
height units ([Z ~> m]) rather than rescaling them to thickness units.  This
change means that the non-Boussinesq open boundary condition calculations avoid
using the Boussinesq reference density.  Associated with this change, 5 internal
variables in update_OBC_segment_data were renamed to reflect that they are layer
vertical extents instead of thicknesses.  Also added or amended comments
describing the purpose and units of 25 real variables in this module and
corrected a handful of lines in this module that were not adhering to the
guidance in the MOM6 style guide.  Answers are bitwise identical in any
Boussinesq cases.  However, answers will change in any non-Boussinesq cases that
use open boundary conditions that use time_interp_extrnal to read in Z-space
data.
Weird errors coming out of parse_perf.  There are unfortunately many
reasons for this, but most likely is that the output format differs
across GitHub nodes.  Perf output is hardly standard.

This will help to diagnose the problem.
@marshallward
Copy link
Owner Author

More or less resolved in dev/gfdl.

@marshallward marshallward deleted the test_perf_output branch May 8, 2024 15:00
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.

None yet

2 participants