Skip to content

Updates for tx1m - #54

Merged
anton-seaice merged 2 commits into
ACCESS-NRI:access_100km_standalonefrom
apcraig:mom100
Sep 28, 2026
Merged

anton-seaice merged 2 commits into
ACCESS-NRI:access_100km_standalonefrom
apcraig:mom100

Conversation

@apcraig

@apcraig apcraig commented Sep 15, 2026

Copy link
Copy Markdown

This does several things for tx1m,

  • Updates the test lists, adds tx1m tests
  • Proposes minor updates to the documentation
  • Adds new decomps for tx1m. This is useful for being able to test bit-for-bit with different block counts and for running on greater varieties of pe counts without running out of blocks.

and a few things discovered during tx1m testing,

  • Fixes some issues in the sumchk unit tester to cover land block elimination better
  • Adds nblock_max (max blocks on a pe across all pes) and nblocks_active (total active blocks) as static module data. These are useful scalars to store in general. We had been just computing them and printing them out but not storing them.

Comment on lines 408 to 409
@@ -407,10 +409,8 @@ subroutine init_domain_distribution(KMTG,ULATG,grid_ice)
#ifdef USE_NETCDF

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think youll need to move the ,& inside the #ifdef, otherwise this won't compiled unless USE_NETCDF is true

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Good catch, I confirmed it's a problem and just pushed a fix.

@anton-seaice

Copy link
Copy Markdown
Collaborator

For future reader, this is related to CICE-Consortium#1132

@anton-seaice

Copy link
Copy Markdown
Collaborator

Thanks @apcraig

@anton-seaice
anton-seaice merged commit 90fb513 into ACCESS-NRI:access_100km_standalone Sep 28, 2026
1 check passed
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