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

fix bug in CURAND.jl's set_stream function. #698

Merged
merged 1 commit into from
Feb 9, 2021
Merged

fix bug in CURAND.jl's set_stream function. #698

merged 1 commit into from
Feb 9, 2021

Conversation

norci
Copy link
Contributor

@norci norci commented Feb 9, 2021

root cause: the parameter stream is not a callable.

root cause: the parameter stream is not a callable.
@norci
Copy link
Contributor Author

norci commented Feb 9, 2021

@maleadt ,
I tried the code in #662 , and got a small error.
I'm not sure if you have fixed it or not.

@codecov
Copy link

codecov bot commented Feb 9, 2021

Codecov Report

Merging #698 (b9c7f91) into master (4eb99b9) will decrease coverage by 0.06%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #698      +/-   ##
==========================================
- Coverage   79.69%   79.62%   -0.07%     
==========================================
  Files         122      122              
  Lines        7356     7356              
==========================================
- Hits         5862     5857       -5     
- Misses       1494     1499       +5     
Impacted Files Coverage Δ
lib/curand/CURAND.jl 96.29% <0.00%> (ø)
lib/curand/random.jl 90.00% <0.00%> (-2.86%) ⬇️
src/pool/binned.jl 94.78% <0.00%> (-0.87%) ⬇️
src/pool.jl 71.60% <0.00%> (-0.83%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4eb99b9...b9c7f91. Read the comment docs.

@maleadt maleadt added the bugfix This gets something working again. label Feb 9, 2021
@maleadt
Copy link
Member

maleadt commented Feb 9, 2021

Thanks!

@maleadt maleadt merged commit 0fb5d76 into JuliaGPU:master Feb 9, 2021
@norci norci deleted the bugfix branch February 9, 2021 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This gets something working again.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants