Skip to content

Commit

Permalink
added TODO(17627): Consider adding randn(BigFloat)/randexp(BigFloat)
Browse files Browse the repository at this point in the history
  • Loading branch information
NHDaly committed Jul 26, 2016
1 parent c4e735d commit 3c91c92
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions base/random.jl
Original file line number Diff line number Diff line change
Expand Up @@ -1194,6 +1194,7 @@ Fill the array `A` with random numbers following the exponential distribution (w
"""
function randexp! end

# TODO(#17627): Consider adding BigFloat support to these functions as well.
let Floats = Union{Float16,Float32,Float64}
for randfun in [:randn, :randexp]
randfun! = Symbol(randfun, :!)
Expand Down
1 change: 1 addition & 0 deletions test/random.jl
Original file line number Diff line number Diff line change
Expand Up @@ -351,6 +351,7 @@ for rng in ([], [MersenneTwister()], [RandomDevice()])
@test_throws MethodError r(Bool)
@test_throws MethodError r(String)
@test_throws MethodError r(AbstractFloat)
# TODO(#17627): Consider adding support for randn(BigFloat) and removing this test.
@test_throws MethodError r(BigFloat)

@test_throws MethodError r(Int64, (2,3))
Expand Down

0 comments on commit 3c91c92

Please sign in to comment.