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

Test base/hdf5_0{3,5}.mpirun={1,4}: do not compare test output #15447

Merged
merged 2 commits into from Jun 23, 2023

Conversation

tamiko
Copy link
Member

@tamiko tamiko commented Jun 23, 2023

We have been adding output variants for this test for years, but the test output is simply not stable. I see different output depending on debug or release configuration and chosen optimization level.

Let us stop here and only check whether the tests run correctly without throwing an exception.

Fixes https://cdash.dealii.org/test/259945

In reference to #15383

We have been adding output variants for this test for years, but the
test output is simply not stable. I see different output depending on
debug or release configuration and chosen optimization level.

Let us stop here and only check whether the test runs correctly without
throwing an exception.
We have been adding output variants for this test for years, but the
test output is simply not stable. I see different output depending on
debug or release configuration and chosen optimization level.

Let us stop here and only check whether the test runs correctly without
throwing an exception.
@tamiko tamiko changed the title Test base/hdf5_03.mpirun=4: do not compare test output Test base/hdf5_0{3,5}.mpirun={1,4}: do not compare test output Jun 23, 2023
@tamiko tamiko added the Tests label Jun 23, 2023
@marcfehling
Copy link
Member

/rebuild

@kronbichler kronbichler merged commit 45d1036 into dealii:master Jun 23, 2023
14 checks passed
@tamiko tamiko deleted the fix_hdf5_test branch July 7, 2023 00:27
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.

None yet

3 participants