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

Kokkos RNG normal() function hangs for small seed value #655

Closed
stanmoore1 opened this issue Feb 24, 2017 · 6 comments
Closed

Kokkos RNG normal() function hangs for small seed value #655

stanmoore1 opened this issue Feb 24, 2017 · 6 comments
Assignees
Labels
Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos)

Comments

@stanmoore1
Copy link
Contributor

stanmoore1 commented Feb 24, 2017

The Kokkos RNG normal() function hangs in its while loop if the RNG seed is a small integer (i.e. 0). It seems like the RNG warmup isn't working properly, or is this expected?

@crtrott
Copy link
Member

crtrott commented Mar 3, 2017

It probably doesn't hang, but it doesn't converge. So we need to something better for warmup. Btw, is it just zero?

@stanmoore1
Copy link
Contributor Author

Yeah, what happened is that our collaborator used the MPI rank as the seed, and for a single core, the seed becomes 0.

@stanmoore1
Copy link
Contributor Author

So even a check on if the seed is valid (i.e. >0) could be helpful.

@mhoemmen
Copy link
Contributor

mhoemmen commented Mar 5, 2017

@crtrott would it make sense just to "time out" of the loop if it does too many iterations (whatever we think "too many" should mean)?

@crtrott
Copy link
Member

crtrott commented Mar 8, 2017

I'll fix it.

@crtrott crtrott self-assigned this Mar 8, 2017
@crtrott crtrott added the Bug Broken / incorrect code; it could be Kokkos' responsibility, or others’ (e.g., Trilinos) label Mar 8, 2017
@crtrott crtrott added this to the 2017-April-end milestone Mar 8, 2017
crtrott added a commit that referenced this issue Mar 21, 2017
@crtrott crtrott closed this as completed Apr 25, 2017
@stanmoore1
Copy link
Contributor Author

Thanks!

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