From be0f8da997e348a8813e0571d4edf3aa133852be Mon Sep 17 00:00:00 2001 From: Avik Pal Date: Fri, 27 Sep 2024 14:52:45 -0400 Subject: [PATCH] fix: __value_derivative removal from line searches --- src/globalization/line_search.jl | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/globalization/line_search.jl b/src/globalization/line_search.jl index 5de4610b6..d43e97247 100644 --- a/src/globalization/line_search.jl +++ b/src/globalization/line_search.jl @@ -110,10 +110,10 @@ function __internal_init( @warn "Scalar AD is supported only for AutoForwardDiff and AutoFiniteDiff. \ Detected $(autodiff). Falling back to AutoFiniteDiff." end - deriv_op = @closure (du, u, fu, p) -> last(__value_derivative( - autodiff, Base.Fix2(f, p), u)) * - fu * - du + deriv_op = @closure (du, u, fu, p) -> begin + # Temporary solution, we are anyways moving to LineSearch.jl + return DI.derivative(f, autodiff, u, Constant(p)) * fu * du + end else # Both forward and reverse AD can be used for line-search. # We prefer forward AD for better performance, however, reverse AD is also supported if user explicitly requests it.