code to use HPE SMARTREDIS

[jedwards4b]

Description of changes

Adds an interface to the HPE SmartSim database. Currently only tested on casper with gnu compiler.
Adds a test mod for testing with smartsim.
Requires CESM_share ESCOMP/CESM_share#5

Provides new xml boolean variables USE_SMARTSIM and CREATE_SMARTSIM_CLUSTER to indicate that this mode should be invoked.

Specific notes

Contributors other than yourself, if any:

CMEPS Issues Fixed (include github issue #):

Are changes expected to change answers?

• bit for bit
• different at roundoff level
• more substantial

Any User Interface Changes (namelist or namelist defaults changes)?

• Yes New variables in drv_in namelist use_smartredis, create_smartsim_cluster
• No

Testing performed if application target is CESM:(either UFS-S2S or CESM testing is required):

• (recommended) CIME_DRIVER=nuopc scripts_regression_tests.py
  • machines:
  • details (e.g. failed tests):
• (recommended) CESM testlist_drv.xml
  • machines and compilers:
  • details (e.g. failed tests):
• (optional) CESM prealpha test
  • machines and compilers
  • details (e.g. failed tests):
• (other) please described in detail
  • machines and compilers
  • details (e.g. failed tests):

Testing performed if application target is UFS-coupled:

• (recommended) UFS-coupled testing
  • description:
  • details (e.g. failed tests):

Testing performed if application target is UFS-HAFS:

• (recommended) UFS-HAFS testing
  • description:
  • details (e.g. failed tests):

Hashes used for testing:

• CESM:
  • repository to check out: https://github.com/ESCOMP/CESM.git
  • branch:
  • hash:
• UFS-coupled, then umbrella repostiory to check out and associated hash:
  • repository to check out:
  • branch:
  • hash:
• UFS-HAFS, then umbrella repostiory to check out and associated hash:
  • repository to check out:
  • branch:
  • hash:

[mvertens]

@jedwards4b - I have asked that output is written to the mediator or driver log file rather than the PET log file to document smartsim usage. Can you also summarize the two new xml variables in the description of the PR.

[mvertens]

Can you please write to the cmeps logunit and not the PET. It would be good to have this in the mediator log file. Or does this belong in the driver write file?

[jedwards4b]

at this point in initialization those logs are not yet available

…

On Tue, Sep 21, 2021 at 9:36 AM mvertens ***@***.***> wrote: ***@***.**** approved this pull request. ------------------------------ In drivers/cime/esmApp.F90 <#213 (comment)>: > + !----------------------------------------------------------------------------- + ! Initialize the CrayLabs SmartRedis client, a stub is provided in share if + ! smartsim is not used. This client shall be used by all component models. + !----------------------------------------------------------------------------- + + if (iam==0) then + open(newunit=fileunit, status="old", file="drv_in") + read(fileunit, smartsim_inparm, iostat=ier) + if (ier > 0) then + call shr_sys_abort('esmApp: error reading in smartsim_inparm namelist from drv_in') + end if + close(fileunit) + end if + call mpi_bcast (use_smartredis, 1, MPI_LOGICAL, 0, COMP_COMM, ier) + call mpi_bcast (create_smartsim_cluster, 1, MPI_LOGICAL, 0, COMP_COMM, ier) + if (use_smartredis) then Can you please write to the cmeps logunit and not the PET. It would be good to have this in the mediator log file. Or does this belong in the driver write file? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#213 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABOXUGHGACZVAV4MBQ2T7T3UDCQ7HANCNFSM5AP5GGEQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

-- Jim Edwards CESM Software Engineer National Center for Atmospheric Research Boulder, CO

[mvertens]

@jedwards4b - what is the status of this PR?

[jedwards4b]

I have a meeting next week with the smartsim developers, they tell me that they have a new method
that will require a different approach from cime. I'll amend or close and replace this at that time.