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

Automatically deactive span based Put API in ADIOS2 when operators are present #1155

Merged
merged 3 commits into from
Dec 30, 2021

Conversation

franzpoeschel
Copy link
Contributor

@franzpoeschel franzpoeschel commented Dec 7, 2021

Ref. this issue
Tldr: Span-based Put API can help save memory, but is incompatible with compression operators

This PR automatically deactivates the span-based Put API on a per-variable basis if the variable has operators defined.
Alternatively, a JSON option can be used to activate / deactivate it completely.

TODO:

@franzpoeschel franzpoeschel force-pushed the automatically-deactive-span-based-api branch 3 times, most recently from d2903ed to 2ad0181 Compare December 9, 2021 13:24
@ax3l ax3l added this to the 0.14.4 milestone Dec 15, 2021
@ax3l
Copy link
Member

ax3l commented Dec 15, 2021

Oh, this cannot be backported to the 0.14.* line if it depends on the JSON PR.

No worries though, we can follow up with 0.15.0 early next year.

@ax3l ax3l removed this from the 0.14.4 milestone Dec 15, 2021
@ax3l ax3l self-requested a review December 15, 2021 23:45
@williamfgc
Copy link

FYI, I'm putting ADIOS2#2981, so an exception is thrown at the back end level.

@ax3l ax3l self-assigned this Dec 20, 2021
@ax3l ax3l force-pushed the automatically-deactive-span-based-api branch from 2ad0181 to 4a80a03 Compare December 21, 2021 01:27
@ax3l
Copy link
Member

ax3l commented Dec 21, 2021

Rebased since #1043 was merged :)

@ax3l ax3l merged commit 05eff02 into openPMD:dev Dec 30, 2021
@ax3l ax3l added this to the 0.14.4 milestone Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants