Handle missing values in clamp

JuliaLang · Feb 14, 2019 · 0d6770d · 0d6770d
1 parent 38d6247
commit 0d6770d
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 0 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -38,6 +38,7 @@ Standard library changes
 * `hasmethod` can now check for matching keyword argument names ([#30712]).
 * `startswith` and `endswith` now accept a `Regex` for the second argument ([#29790]).
 * `retry` supports arbitrary callable objects ([#30382]).
+* `clamp` can now handle missing values ([#31065]).
 
 #### LinearAlgebra
 

diff --git a/base/math.jl b/base/math.jl
@@ -1070,5 +1070,6 @@ for f in (:(acos), :(acosh), :(asin), :(asinh), :(atan), :(atanh),
           :(log2), :(exponent), :(sqrt))
     @eval $(f)(::Missing) = missing
 end
+clamp(::Missing, lo, hi) = missing
 
 end # module
diff --git a/test/math.jl b/test/math.jl
@@ -27,6 +27,8 @@ end
         clamp!(x, 1, 3)
         @test x == [1.0, 1.0, 2.0, 3.0, 3.0]
     end
+
+    @test ismissing(clamp(missing, 1, 2))
 end
 
 @testset "constants" begin