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

Add an example: estimating pi using Monte-Carlo method #26

Merged
merged 10 commits into from
Oct 19, 2020

Conversation

tkf
Copy link
Member

@tkf tkf commented Oct 18, 2020

Commit Message

Add an example: estimating pi using Monte-Carlo method (#26)

Add an example for combining FoldsCUDA and counter-based RNG in
Random123. This requires minor fixes in FoldsCUDA and Transducers
that was expecting the length(_) to be an Int which is not the
case for StepRange{UInt64, UInt64} returned by counters(n) used in
the example.

@codecov
Copy link

codecov bot commented Oct 18, 2020

Codecov Report

Merging #26 into master will decrease coverage by 2.37%.
The diff coverage is 46.15%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
- Coverage   66.23%   63.85%   -2.38%     
==========================================
  Files           7        9       +2     
  Lines         154      166      +12     
==========================================
+ Hits          102      106       +4     
- Misses         52       60       +8     
Impacted Files Coverage Δ
examples/monte_carlo_pi.jl 36.36% <36.36%> (ø)
src/kernels.jl 49.52% <100.00%> (-0.96%) ⬇️
test/nogpu/test_monte_carlo_pi.jl 100.00% <100.00%> (ø)

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 fa2382f...2320c87. Read the comment docs.

@tkf
Copy link
Member Author

tkf commented Oct 19, 2020

CPU/threading requires JuliaFolds/Transducers.jl#433

@tkf tkf changed the title Add monte_carlo_pi.jl Add an example: estimating pi using Monte-Carlo method Oct 19, 2020
@tkf tkf marked this pull request as ready for review October 19, 2020 00:33
@mergify mergify bot merged commit 879451c into master Oct 19, 2020
@mergify mergify bot deleted the monte_carlo_pi branch October 19, 2020 03:24
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.

1 participant