Skip to content

Conversation

wlin7
Copy link
Contributor

@wlin7 wlin7 commented Jun 2, 2025

This PR adding the following grids with ne256pg2 atm and RRSwISC6to18E3r5 mpas meshes

  • ne256pg2_r025_RRSwISC6to18E3r5
  • ne256pg2_r0125_RRSwISC6to18E3r5
  • ne256pg2_RRSwISC6to18E3r5

Default topo for ne256pg2 is replaced. A new ncdata for L80 ne256pg2 is also specified.

[NML]
[BFB] for all currently tested configurations


  • Tests are underway with F2010 compset and ne256pg2_r025_RRSwISC6to18E3r5
  • Using updated domain.lnd.r025_RRSwISC6to18E3r5.250216.nc
  • elmi.CNPRDCTCBCTOP.ne120pg2_r025_RRSwISC6to18E3r5.1985.nc not yet made default for finidat

Remaining issue and a tentative workaround

  • Many incidents of lminval, lmaxval, ominval, omaxval exceeding tolerance limit of eps_fracval=1e-2 (minval < -1e-2 or maxval > 1.01), examples (print out of both minval and maxval where epslos were hit:
  • lminval & lmaxval = -1.516605360041501E-003 1.01150894379774
  • lminval & lmaxval = -1.304605429039385E-002 1.00031458099706
  • ominval & omaxval = -2.081773763604969E-002 1.00000000000027
  • ominval & omaxval = -2.134231120258579E-008 1.02282679018699

The tentative workaround to allow EAMv3 ne256 testbed effort to proceed:

  • Increase eps_fracval = 3.0e-02 in driver-mct/main/seq_frac_mct.F90
  • ./xmlchange EPS_FRAC=3E-2 does not take effect

With updated map_RRSwISC6to18E3r5_to_ne256pg2_traave.20250618.nc (via 86d001e), no more exceeding eps_fracval of 1e-02.

@wlin7 wlin7 force-pushed the wlin/ne256pg2_RRSwISC6to18E3r5 branch from 3015ebd to 9bf394e Compare June 2, 2025 21:14
@rljacob
Copy link
Member

rljacob commented Jun 4, 2025

Why is this labeled "Do not merge" ?

@wlin7
Copy link
Contributor Author

wlin7 commented Jun 4, 2025

Why is this labeled "Do not merge" ?

Put it up for testing with EAMv3 and discussion. There are still runtime issues being debugged on.

@rljacob
Copy link
Member

rljacob commented Jun 4, 2025

Then lets convert this to draft.

@rljacob rljacob marked this pull request as draft June 4, 2025 16:00
@wlin7 wlin7 requested a review from rljacob June 4, 2025 17:05
@wlin7
Copy link
Contributor Author

wlin7 commented Jun 4, 2025

Supposed to show "Convert to draft" link just below the Reviewers but couldn't find it.
@rljacob , can you instruct how to do the convert?

@wlin7
Copy link
Contributor Author

wlin7 commented Jun 4, 2025

Thanks, Rob. Just realized you already converted it.

@wlin7 wlin7 marked this pull request as ready for review June 26, 2025 14:59
@wlin7 wlin7 requested review from jonbob June 26, 2025 15:00
@jonbob
Copy link
Contributor

jonbob commented Jun 26, 2025

@wlin7 -- the following files have permissions set to -rw-rwSr--

  • cpl/gridmaps/ne256pg2/map_ne256pg2_to_r025_trfvnp2.20250521.nc
  • cpl/gridmaps/ne256pg2/map_ne256pg2_to_r025_trbilin.20250521.nc
  • cpl/gridmaps/r025/map_r025_to_ne256pg2_traave.20250521.nc
  • cpl/gridmaps/ne256pg2/map_ne256pg2_to_r025_traave.20250521.nc
  • cpl/gridmaps/ne256pg2/map_ne256pg2_to_RRSwISC6to18E3r5_traave.20250414.nc
  • cpl/gridmaps/ne256pg2/map_ne256pg2_to_RRSwISC6to18E3r5_trbilin.20250415.nc
  • cpl/gridmaps/ne256pg2/map_ne256pg2_to_RRSwISC6to18E3r5-nomask_trbilin.20250415.nc
  • cpl/gridmaps/ne256pg2/map_ne256pg2_to_RRSwISC6to18E3r5_trfvnp2.20250415.nc

But otherwise I verified that all new files are in the inputdata repo and either 64-bit offset or classic

@jonbob
Copy link
Contributor

jonbob commented Jul 1, 2025

@wlin7 -- I pushed a commit adding lnd<=>rof mapping files required for the ne256pg2_RRSwISC6to18E3r5 configuration. You had already made the files, so they just needed to be specified in config_grids

@wlin7
Copy link
Contributor Author

wlin7 commented Jul 14, 2025

Thanks for the careful checks, @jonbob. The group permission bit for the map files have been reset. The missing map_ne256pg2_to_r0125_trfvnp2.20250415.nc has been added. Thanks also for adding the ROF2LND and LND2ROF mapping files for the bigrid.

Copy link
Contributor

@jonbob jonbob left a comment

Choose a reason for hiding this comment

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

Verified the missing file is now available and permissions are correct. Approved based on testing and visual inspection

@jonbob jonbob added the BFB PR leaves answers BFB label Jul 14, 2025
jonbob added a commit that referenced this pull request Jul 14, 2025
New grids with ne256pg2 atm and supporting files

This PR adding the following grids with ne256pg2 atm and
RRSwISC6to18E3r5 mpas meshes:
* ne256pg2_r025_RRSwISC6to18E3r5
* ne256pg2_r0125_RRSwISC6to18E3r5
* ne256pg2_RRSwISC6to18E3r5
Default topo for ne256pg2 is replaced. A new ncdata for L80 ne256pg2 is
also specified.

[BFB] for all currently tested configurations
@jonbob
Copy link
Contributor

jonbob commented Jul 14, 2025

merged to next

@jonbob jonbob merged commit 2237a9c into master Jul 15, 2025
7 checks passed
@jonbob jonbob deleted the wlin/ne256pg2_RRSwISC6to18E3r5 branch July 15, 2025 17:02
@jonbob
Copy link
Contributor

jonbob commented Jul 15, 2025

merged to master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB Coupled Model
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants