Skip to content

Julienne multi-image parallel callbacks (issue #57)#157

Merged
bonachea merged 5 commits into
BerkeleyLab:mainfrom
bonachea:parallel-callbacks
Dec 15, 2025
Merged

Julienne multi-image parallel callbacks (issue #57)#157
bonachea merged 5 commits into
BerkeleyLab:mainfrom
bonachea:parallel-callbacks

Conversation

@bonachea

@bonachea bonachea commented Dec 14, 2025

Copy link
Copy Markdown
Member

Parameterize Julienne's multi-image support to allow expansion to
other SPMD multi-process models beyond Fortran's built-in SPMD features.

Now when HAVE_MULTI_IMAGE_SUPPORT is enabled and JULIENNE_PARALLEL_CALLBACKS
is defined, Julienne will no longer directly invoke the Fortran
multi-image features. Instead it will invoke client-provided procedure
pointer callbacks that implement the necessary features.

This allows the client to swap in another SPMD multi-process interface,
for example PRIF or MPI.

Fixes issue #57

@rouson rouson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Thanks, @bonachea!

There are zero content changes in this commit
Parameterize Julienne's multi-image support to allow expansion to
other SPMD multi-process models beyond Fortran's built-in SPMD features.

Now when HAVE_MULTI_IMAGE_SUPPORT is enabled and JULIENNE_PARALLEL_CALLBACKS
is defined, Julienne will no longer directly invoke the Fortran
multi-image features. Instead it will invoke client-provided procedure
pointer callbacks that implement the necessary features.

This allows the client to swap in another SPMD multi-process interface,
for example PRIF or MPI.

Fixes issue BerkeleyLab#57
Also tweak ifx to provide better crash output
@bonachea bonachea merged commit f96efdb into BerkeleyLab:main Dec 15, 2025
28 checks passed
@bonachea bonachea deleted the parallel-callbacks branch December 15, 2025 18:03
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