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

Combine all global sums in SIS_sum_output #128

Merged
merged 2 commits into from
Aug 8, 2020

Conversation

Hallberg-NOAA
Copy link
Member

Revised SIS_sum_output.F90 to merge all blocking global sums in SIS2 that are
used routinely into a single blocking sum in SIS_sum_output for improved model
efficiency. Also removed unneeded variables from the SIS_sum_out_CS and
reformatted the module use statements in this module. All answers are bitwise
identical.

  Revised SIS_sum_output.F90 to merge all blocking global sums in SIS2 that are
used routinely into a single blocking sum in SIS_sum_output for improved model
efficiency.  Also removed unneeded variables from the SIS_sum_out_CS and
reformatted the module use statements in this module.  All answers are bitwise
identical.
  Revised SIS_sum_output.F90 to reproduce previous non-reproducing sums of total
mass, salt and heat between hemispheres.  These values do not alter solutions or
SIS_parameter_doc files, but they were reported to stdout.  Also provide array
extents over which to do sums to reproducing_sum_EFP calls, avoiding working on
zero values in halo points. All solutions are bitwise identical.
@marshallward
Copy link
Member

This has passed my internal Gaea regression.

@marshallward marshallward merged commit a4e4f4d into NOAA-GFDL:dev/gfdl Aug 8, 2020
@Hallberg-NOAA Hallberg-NOAA deleted the merge_global_sums branch August 26, 2022 20: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.

2 participants