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

Serial unit test failure #1632

Closed
ibaned opened this issue May 22, 2018 · 3 comments
Closed

Serial unit test failure #1632

ibaned opened this issue May 22, 2018 · 3 comments
Assignees
Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)
Milestone

Comments

@ibaned
Copy link
Contributor

ibaned commented May 22, 2018

@prwolfe wrote the following as trilinos/Trilinos#2793 :

Kokkos unit test failing with MPICH

When building with MPICH3.2 I see a failure in KokkosCore_UnitTest_Serial.exe (the serial.cxx test) due to comparison of two doubles. It looks like they are off by less than 10-9 and I'm not sure what if any tolerance is acceptable in this case.

@trilinos/kokkos

Expectations

This test should pass with all mpi variants

Current Behavior

This test passes in openmpi builds

Steps to Reproduce

To recreate this you can use the PR template we are working on. I did attempt to recreate this In a kooks stand-alone build without sucess.

source ${TrilinosSource}/cmake/std/sems/PullRequestIntel17.0.1TestingEnv.sh
cmake -C ${TrilinosSource}/cmake/std/PullRequestLinuxIntelTestingSettings.cmake -DTrilinos_ENABLE_Kokkos=ON ${TrilinosSource}
cd pacakges/kokkos/core/unit_tests
make
@ibaned ibaned added the Question For Kokkos internal and external contributors and users label May 22, 2018
@ibaned ibaned added this to the 2018 July milestone May 22, 2018
@ibaned ibaned self-assigned this May 22, 2018
@prwolfe
Copy link

prwolfe commented May 22, 2018 via email

@ibaned
Copy link
Contributor Author

ibaned commented Aug 16, 2018

@prwolfe I think I fixed this will pull request #1729. It doesn't have so much to do with MPICH as opposed to how the compiler optimizes to different ways of computing the same number. This fix should show up in the next Kokkos release.

@ibaned ibaned added InDevelop Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos) and removed Question For Kokkos internal and external contributors and users labels Aug 16, 2018
@prwolfe
Copy link

prwolfe commented Aug 16, 2018

Thanks @ibaned ! I will watch for it.

@crtrott crtrott closed this as completed Nov 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)
Projects
None yet
Development

No branches or pull requests

3 participants