-
-
Notifications
You must be signed in to change notification settings - Fork 173
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
Make ADIOS tests compatible with 2.9 and 2.10 #3128
Conversation
We know we are not compatible and will do a minor DOLFINx release once ADIOS 2.10 is widely available.
I'll add a try except in the test, give me a few minutes to build adios 2.10 locally to make a fix |
@jhale I added a patch to the test. As the only thing that doesn't work with adios2 is the test, i think it is better to patch the test for backwards compatibility than restricting build versions |
Why not apply fixes to be compatible with 2.10? How much work would it be? |
Ive changed the scope of the PR. We are now compatible with 3.10, given that try except clause in the adios tests (only Python interface changed from 2.9 to 2.10). |
Looks good |
We know we are not compatible with 2.10. Will do a minor DOLFINx release once ADIOS 2.10 is widely available.