Skip to content

Commit 49c7f2f

Browse files
Fix deprecated field names in MultistartOptimization result handling
Updates the `_local_optimiser` function in OptimizationMultistartOptimization to use the current field names for optimization results: - Changed `res.minimum` to `res.objective` - Changed `res.minimizer` to `res.u` This resolves deprecation warnings when running tests and ensures compatibility with the current OptimizationSolution API. The traits for MultistartOptimization.TikTak are already correctly set: - `requiresbounds = true` (TikTak needs bounds for Sobol sequence generation) - `allowsbounds = true` (TikTak passes bounds to local optimizer) Fixes issue mentioned in https://discourse.julialang.org/t/error-when-using-multistart-optimization/133174 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent d201417 commit 49c7f2f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/OptimizationMultistartOptimization/src/OptimizationMultistartOptimization.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ function SciMLBase.__solve(cache::OptimizationBase.OptimizationCache{
6262
_local_optimiser = function (pb, θ0, prob)
6363
prob_tmp = remake(prob, u0 = θ0)
6464
res = solve(prob_tmp, cache.solver_args.local_opt)
65-
return (value = res.minimum, location = res.minimizer, ret = res.retcode)
65+
return (value = res.objective, location = res.u, ret = res.retcode)
6666
end
6767

6868
local_optimiser(pb, θ0) = _local_optimiser(pb, θ0, cache.solver_args.prob)

0 commit comments

Comments
 (0)