Skip to content

Conversation

franzpoeschel
Copy link
Contributor

@franzpoeschel franzpoeschel commented Jul 16, 2025

Workaround for this bug ornladios/ADIOS2#4586.
Since BP5 can write directly from user buffers, this change (essentially activating our fallback implementation) is not too costly.

TODO:

  • Wait for an answer in the issue linked above. The current draft assumes that this will be fixed in upcoming versions, but let's be sure of that.

@franzpoeschel franzpoeschel force-pushed the deactivate-span-bp5-by-default branch from 575bec9 to ef2042f Compare July 17, 2025 08:34
@franzpoeschel franzpoeschel enabled auto-merge (squash) July 17, 2025 14:54
@franzpoeschel franzpoeschel requested a review from ax3l July 17, 2025 14:55
@franzpoeschel franzpoeschel merged commit 0263879 into openPMD:dev Jul 17, 2025
30 checks passed
@ax3l ax3l requested a review from guj September 2, 2025 17:19
@guj
Copy link
Contributor

guj commented Sep 2, 2025

This bug was fixed in BP5. Should span be allowed again in BP5?

@franzpoeschel
Copy link
Contributor Author

This bug was fixed in BP5. Should span be allowed again in BP5?

It is allowed based on the discovered ADIOS2 version, any version newer than 2.10.2 will use the Span API by default again:

#if (                                                                          \
    ADIOS2_VERSION_MAJOR * 1000 + ADIOS2_VERSION_MINOR * 10 +                  \
    ADIOS2_VERSION_PATCH) <= 2102
            ||
            // Deactivate the Span API in BP5 by default due to this bug
            // https://github.com/ornladios/ADIOS2/issues/4586,
            // fixed by https://github.com/ornladios/ADIOS2/pull/4587
            this->realEngineType() == "bp5"
#endif

Do you know if the error you saw is related to this?

@guj
Copy link
Contributor

guj commented Sep 2, 2025

The one from master is 2.10.0.735 from bpls -V
so that's why span was deactivated.

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