Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

load_inputs_file: Simplify File Check #704

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Sep 17, 2024

AMReX will now check this in a user-friendly and performant manner.

AMReX will now check this in a user-friendly and performant manner.
@ax3l ax3l added component: third party Changes in ImpactX that reflect a change in a third-party library component: python Python bindings labels Sep 17, 2024
WeiqunZhang pushed a commit to AMReX-Codes/amrex that referenced this pull request Sep 18, 2024
## Summary

If a file added via `ParmParse:addFile` does not exist, we did not yet
receive a user-friendly error message. This fixes this in a way that
does not hammer the file system from all MPI ranks.

## Additional background

Follow-up to #2842 #2936 #3440

X-ref: ECP-WarpX/WarpX#5283
ECP-WarpX/impactx#704
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: python Python bindings component: third party Changes in ImpactX that reflect a change in a third-party library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant