Skip to content

Add a dm_info object to the argument list of core mpas_setup_packages_function - #1463

Merged
mgduda merged 1 commit into
MPAS-Dev:developfrom
mgduda:setup_packages_with_dminfo
Jun 2, 2026
Merged

Add a dm_info object to the argument list of core mpas_setup_packages_function#1463
mgduda merged 1 commit into
MPAS-Dev:developfrom
mgduda:setup_packages_with_dminfo

Conversation

@mgduda

@mgduda mgduda commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

This PR adds a dm_info object to the argument list of the mpas_setup_packages_function core interface function. Adding a dm_info object allows a core's package setup function to make use of MPI when setting up packages. For example, a core may read data from a file only on MPI rank 0 and broadcast the resulting values to other MPI ranks rather than having each MPI rank independently open and read the file.

In addition to updating the mpas_setup_packages_function abstract interface in mpas_core_types.inc with an additional dminfo argument, this PR updates the call to domain_ptr % core % setup_packages in mpas_init to include the dminfo argument, and it also updates the *_setup_packages functions in each core's interface module.

…_function

Adding a dm_info object allows a core's package setup function to make use of
MPI when setting up packages. For example, a core may read data from a file only
on MPI rank 0 and broadcast the resulting values to other MPI ranks rather than
having each MPI rank independently open and read the file.

In addition to updating the mpas_setup_packages_function abstract interface in
mpas_core_types.inc with an additional 'dminfo' argument, this commit updates
the call to domain_ptr % core % setup_packages in mpas_init to include the
dminfo argument, and it also updates the *_setup_packages functions in each
core's interface module.
@mgduda
mgduda requested a review from abishekg7 June 1, 2026 23:10

@abishekg7 abishekg7 left a comment

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.

Looks good. Checked with gnu, intel and nvhpc builds. And also with atmosphere, init_atmosphere, sw cores.

@mgduda
mgduda merged commit 5191ccc into MPAS-Dev:develop Jun 2, 2026
@mgduda
mgduda deleted the setup_packages_with_dminfo branch June 2, 2026 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants