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

Fix bug when writing null blocks #3542

Merged
merged 1 commit into from
Mar 17, 2023

Conversation

anagainaru
Copy link
Contributor

@anagainaru anagainaru commented Mar 17, 2023

This solves a fix for WarpX where processes write different number of blocks.

Related to PR #3465 and issue #3460

Note: This only affects BP4

Co-authored-by: lizdulac <lizadulac@gmail.com>
Co-authored-by: Norbert Podhorszki <pnorbert@ornl.gov>
@anagainaru anagainaru requested review from pnorbert and lizdulac March 17, 2023 15:14
@anagainaru anagainaru changed the base branch from master to release_29 March 17, 2023 15:15
@anagainaru
Copy link
Contributor Author

@ax3l I don't think we need #3465, the copyToBuffer function should never receive 0 size arrays

@anagainaru anagainaru added this to the v2.9.0 milestone Mar 17, 2023
@vicentebolea
Copy link
Collaborator

@anagainaru feel free to merge when ready

@anagainaru anagainaru merged commit 8a6ae91 into ornladios:release_29 Mar 17, 2023
vicentebolea pushed a commit that referenced this pull request Mar 20, 2023
@anagainaru anagainaru deleted the warpx-bug-null-blocks branch March 30, 2023 15:49
@ax3l
Copy link
Contributor

ax3l commented May 4, 2023

Thank you so much @anagainaru , will try again!

@ax3l
Copy link
Contributor

ax3l commented Aug 28, 2023

@anagainaru should we add a test for this in ADIOS2? (Zero writers and zero writers w/ compressions.)

@ax3l
Copy link
Contributor

ax3l commented Oct 26, 2023

@anagainaru ping on test coverage? :) #3792

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.

4 participants