Skip to content

Commit 404f4f3

Browse files
Merge pull request #1 from ParasPuneetSingh/ParasPuneetSingh-patch-1
Update OptimizationOptimJL.jl
2 parents 038c7b6 + 01f4479 commit 404f4f3

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

lib/OptimizationOptimJL/src/OptimizationOptimJL.jl

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ SciMLBase.requiresbounds(opt::Optim.SAMIN) = true
1414
SciMLBase.supports_opt_cache_interface(opt::Optim.AbstractOptimizer) = true
1515
SciMLBase.supports_opt_cache_interface(opt::Union{Optim.Fminbox, Optim.SAMIN}) = true
1616
SciMLBase.supports_opt_cache_interface(opt::Optim.ConstrainedOptimizer) = true
17+
SciMLBase.requiresderivative(Optim.AbstractOptimizer) = !(opt isa Optim.ZerothOrderOptimizer)
1718

1819
function __map_optimizer_args(cache::OptimizationCache,
1920
opt::Union{Optim.AbstractOptimizer, Optim.Fminbox,
@@ -128,7 +129,8 @@ function SciMLBase.__solve(cache::OptimizationCache{
128129
local x, cur, state
129130

130131
cur, state = iterate(cache.data)
131-
132+
(requiresderivative(Optim.AbstractOptimizer))
133+
error("Use OptimizationFunction to pass the derivatives or automatically generate them with one of the autodiff backends")
132134
!(cache.opt isa Optim.ZerothOrderOptimizer) && cache.f.grad === nothing &&
133135
error("Use OptimizationFunction to pass the derivatives or automatically generate them with one of the autodiff backends")
134136

0 commit comments

Comments
 (0)