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

ADIOS2: Support NULL-Blocks (Chunks) #355

Merged
merged 1 commit into from
Feb 2, 2023

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Feb 2, 2023

Add support to read back ADIOS2 data sets that contain empty blocks (NULL chunks). This is something to handle carefully and needed for a work-arounds in WarpX BTD.

X-Refs.:

Add support to read back ADIOS2 data sets that contain
empty blocks (NULL chunks). This is something to handle
carefully and needed for a work-arounds in WarpX BTD.
@RemiLehe RemiLehe merged commit bbc5d6b into openPMD:dev Feb 2, 2023
@ax3l ax3l deleted the fix-support-null-blocks branch February 2, 2023 18:07
@@ -66,6 +66,15 @@ def get_data(series, record_component, i_slice=None, pos_slice=None,
data = np.full(record_component.shape, np.nan, record_component.dtype)
for chunk in chunks:
chunk_slice = chunk_to_slice(chunk)

# skip empty slices
# https://github.com/ornladios/ADIOS2
Copy link
Member Author

Choose a reason for hiding this comment

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

Should read ornladios/ADIOS2#3459

Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -98,6 +107,15 @@ def get_data(series, record_component, i_slice=None, pos_slice=None,
s_valid = list(s) # same as s but reduced to valid regions in chunk
s_target = [] # starts and stops in sliced array
chunk_slice = chunk_to_slice(chunk)

# skip empty slices
# https://github.com/ornladios/ADIOS2
Copy link
Member Author

Choose a reason for hiding this comment

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

Should read ornladios/ADIOS2#3459

Copy link
Member Author

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants