Add trivial implementation for `prif_fail_image` by bonachea · Pull Request #289 · BerkeleyLab/caffeine · GitHub
Skip to content

Add trivial implementation for prif_fail_image#289

Merged
bonachea merged 4 commits into
BerkeleyLab:mainfrom
bonachea:fail_image
Jan 28, 2026
Merged

Add trivial implementation for prif_fail_image#289
bonachea merged 4 commits into
BerkeleyLab:mainfrom
bonachea:fail_image

Conversation

@bonachea

Copy link
Copy Markdown
Member

This provides a trivial implementation of prif_fail_image() that just hangs the calling image forever (unless it is the only image, in which case it just terminates). This is "trivial" in the sense that other images never directly learn that another image has failed, but it is nevertheless believed to be standards-conforming.

@bonachea bonachea requested a review from ktras January 20, 2026 03:35

@ktras ktras 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.

LGTM but I have a suggested improvement to an inline code comment.

Comment thread example/support-test/fail_image.F90
@bonachea bonachea requested a review from ktras January 27, 2026 18:56

@ktras ktras 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.

LGTM

This provides a trivial implementation of prif_fail_image() that
just hangs the calling image forever (unless it is the only image,
in which case it just terminates). This is "trivial" in the sense
that other images never directly learn that another image has failed,
but it is nevertheless believed to be standards-conforming.
@bonachea bonachea enabled auto-merge January 28, 2026 18:17
@bonachea bonachea merged commit 4388136 into BerkeleyLab:main Jan 28, 2026
34 checks passed
@bonachea bonachea deleted the fail_image branch January 28, 2026 18:38
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