Skip to content

Conversation

@vkarak
Copy link
Contributor

@vkarak vkarak commented Sep 3, 2019

Fixes MAINT-107.

@ajocksch Strangely the test fails with PrgEnv-gnu with the following error:

 MPI test on CPU using           12 tasks
 Result : FAIL
 Expected value sum(f3):    110.000000     actual value:   55.0000000

Copy link
Contributor

@teojgo teojgo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm although I get the same failure with PrgEnv-gnu

@vkarak
Copy link
Contributor Author

vkarak commented Sep 3, 2019

The test is not passing even on Daint with PrgEnv-gnu. I guess sth has changed in the test files. I'm doing git bisect to locate the exact commit now.

@vkarak
Copy link
Contributor Author

vkarak commented Sep 3, 2019

This commit broke the test: 8623851

@lucamar
Copy link
Contributor

lucamar commented Sep 3, 2019

@vkarak Merging #913 should bring the check back to its previous state

@vkarak
Copy link
Contributor Author

vkarak commented Sep 3, 2019

Thanks, @lucamar. We will merge this after then.

@vkarak
Copy link
Contributor Author

vkarak commented Sep 3, 2019

This PR is not needed anymore, since the problem is fixed in #913.

@vkarak vkarak closed this Sep 3, 2019
@vkarak vkarak deleted the test/fix-openacccudacpp-test branch September 3, 2019 16:47
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