Skip to content

Conversation

@mlee03
Copy link
Contributor

@mlee03 mlee03 commented Dec 31, 2025

Description
This PR updates the horiz_interp_dev branch

Fixes # (issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note
any relevant details for your test configuration (e.g. compiler, OS). Include
enough information so someone can reproduce your tests.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • New check tests, if applicable, are included
  • make distcheck passes

abrooks1085 and others added 7 commits December 8, 2025 12:55
…OAA-GFDL#1775)

* Refactor MPP_GATHER_PELIST_3D_ to implement 3D gathers using the existing MPP_GATHER_1D(V)_ routines, leveraging collective communication instead of manual point-to-point sends/recvs.

* Remove trailing whitespace
Co-authored-by: vithikashah001 <[email protected]>
* Refactor of MPI-IO support in fms2_io

MPI-IO support in fms2_io has been refactored.
`open_collective_netcdf_file` has been removed and its functionality has
been merged into `netcdf_file_open`. Parallel reads and writes are now
enabled using the same interface and code pathways.

* Specify argument names in `netcdf_file_open` call

Use explicit names for the optional arguments of `netcdf_file_open`,
when it's called within `open_domain_file`.

* netcdf_read_data_*: Call nf90_var_par_access later

In the netcdf_read_data_* subroutines, move the calls to
nf90_var_par_access so that they run later.

* Fix domain reads when MPI-IO is enabled

Update the domain read subroutines so that they set the indices
correctly when use_netcdf_mpi is enabled.

* Remove fallback test for collective reads

Remove the test which attempts to open a 64-bit NetCDF file in
collective read mode. Attempting to do so will now fail rather than fall
back to a standard read without MPI-IO.

* Restore working version of netcdf_read_data.inc

Undo changes to netcdf_read_data.inc which were committed
unintentionally.

* Update call to open_file in diag_manager

* Restore collective read unit test

Restore the unit test which attempts to open a 64-bit NetCDF file for MPI-IO
collective reads. Change from `test_expect_success` to `test_expect_failure`.

* Explain MPI-IO features in README

Add an explanation of fms2_io's MPI-IO support to the readme.md file.

---------

Co-authored-by: Jesse Lentz <[email protected]>
Co-authored-by: Jesse Lentz <[email protected]>
@mlee03 mlee03 changed the base branch from main to horiz_interp_dev December 31, 2025 18:02
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.

6 participants