Conversation
Update AGENTS.md to specify what compsets and resolutions to possibly create to test code changes. Also discourage trying make a case or build if not on a supported machine.
Update development workflow information. Remove old e3sm.org URL for supported machines. Add examples of query_config commands to get supported machine and compset info
Co-authored-by: ndkeen <5684727+ndkeen@users.noreply.github.com>
Co-authored-by: ndkeen <5684727+ndkeen@users.noreply.github.com>
Mam4xx has two constants: pcnst=40 and gas_pcnst=31. Array was dimentioned by the second but indexed by the first. [BFB]
Update the physical constants files with the latest changes from the PCD repository.
Change the list of github workflows used for CI/CD. Explain how to get more info about test suites.
Add an AGENTS.md file to help provide context to AI programming tools. Also make sure Claude Code, which is almost alone in ignoring it, uses it by using a hidden CLAUDE.md file. copilot will use AGENTS.md but will also try to make its own so have it also include AGENTS.md so everyone has the same context. Add a .fortls file for the Fortran Language Server Closes #8086 [BFB]
Adding support for data compression using HDF5. Also, adding support for initializing the MPI library with thread support. This change also integrates a new version of SCORPIO (1.9.2). * Updating CIME build system to support HDF5 (with and without data compression support) * Lossless data compression support using Blosc2 + Zstd and lossy compression support using ZFP * Adding HDF5 compression libraries for Frontier, Perlmutter and ANL compute nodes * Updating MPAS framework to support HDF5 * Updating to SCORPIO version 1.9.2 SCORPIO version 1.9.2 includes, * Multiple fixes related to data compression * Misc bug fixes [BFB]
The identification of the vertices that are common between two grid cells is fixed by correcting the starting value in the second do-loop. Additionally, the MOAB-related information is now created in ELM if the code is compiled with -DHAVE_MOAB flag. [BFB]
Remove `-fbacktrace` from GNU C/C++ debug flags
It was using the wrong macro, causing a warning and failing to add the tests to the desired labels
Add limited support for new grids with a few aliases. The new aliases were intended to support a PPE ensemble, so the combination of land and ocean grids will not be ideal for other purposes, but other users can build off these changes to add new aliases. Also, while these changes include new atmosphere initial conditions I have omitted new new drydep_srf_file entries, which are needed to run EAMv3 on these grids. [BFB]
Author
Author
TestingCTest unit tests:
Polaris
|
Author
Use the right macro, avoiding a cmake warning, and ensuring the tests get the desired label. [BFB]
…(PR #8162) Port zm_conv_mcsp_calculate_shear Ports zm_conv_mcsp_calculate_shear in zm_conv_mcsp.F90. I think this is the only subroutine that needs to be ported from this file. Also, removes the state_v calculations from zm_conv_mcsp_calculate_shear. If you look at the subroutine, the only output is happening here mcsp_shear(i) = storm_u_shear(i), so the _v stuff was unused. [BFB] * jgfouca/port_zm_conv_mcsp_calculate_shear: Fix remove unused zmid Changes based on copilot review Remove unused var Fixes and remove state_v stuff that was never used Parallelize index search Progress, but diffs AI attempt Port zm_conv_mcsp_calculate_shear
Fix non bfb log calls in f90 entropy. This should fix remaining CI issues once merged and baselines are regenerated. [BFB]
10 tasks
Author
|
Building with gnugpu on Perlmutter, I'm seeing the errors reported in #364. I'll need to follow up on that. I don't think it's an issue of wrong modules but I could be wrong. |
Deep in ZM, some f90 needed to use bfb math functions. [BFB]
Fix some subtle non-bfbness with f90 Deep in ZM, some f90 needed to use bfb math functions. Also, make bfb math stuff callable from pure/elemental fortran functions. This should fix the last CI ZM problem. [BFB]
Collaborator
|
I merged #360 and this PR into Polaris
|
Author
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This brings in module updates on Aurora and Perlmutter as well as SCORPIO (among others):
E3SM-Project#8109
E3SM-Project#8113
E3SM-Project#7996
Checklist
Testingwith the following:have been run on and indicate that are all passing.
has passed, using the Polaris
e3sm_submodules/Omegabaseline-pfor both the baseline (Polarise3sm_submodules/Omega) and the PR build