Merge pull request #120 from jClugstor/scimlsensitivity_bug_fix

FixedSizeDiffCache use `Nothing` instead of `nothing`

Author: ChrisRackauckas

src/PreallocationTools.jl

@@ -11,7 +11,7 @@ end
 function FixedSizeDiffCache(u::AbstractArray{T}, siz,
         ::Type{Val{chunk_size}}) where {T, chunk_size}
     x = ArrayInterface.restructure(u,
-        zeros(ForwardDiff.Dual{nothing, T, chunk_size},
+        zeros(ForwardDiff.Dual{Nothing, T, chunk_size},
             siz...))
     xany = Any[]
     FixedSizeDiffCache(deepcopy(u), x, xany)

test/core_dispatch.jl

@@ -164,3 +164,12 @@ b = PreallocationTools.DiffCache(zeros(4), Val{4}())
 c = PreallocationTools.DiffCache(zeros(4), Val{4})
 @test structequal(a, b)
 @test structequal(a, b)
+
+
+# FixedSizeDiffCache get_tmp ReinterpretArray test
+# Ensures that get_tmp doesn't produce a Reinterpret array in some cases
+
+dual_cache = FixedSizeDiffCache([0.0, 0.0, 0.0], 3)
+dl = zeros(ForwardDiff.Dual{Nothing, Float64, 3}, 3)
+
+@test !(get_tmp(dual_cache, dl) isa Base.ReinterpretArray)