SciML/Optimization.jl

Add `sense` support for all algorithms

Offen

#976 geöffnet am 01.06.2023

 (4 Kommentare) (1 Reaktion) (1 zugewiesene Person)Julia (99 Forks)batch import
good first issue

Repository-Metriken

Stars
 (824 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 2T 1h) (20 gemergte PRs in 30 T)

Beschreibung

Currently sense is only supported for a subset of algorithms. This breaks the common interface as some solvers solve min vs max. Furthermore, since sense is a kwarg for OptimizationProblem and not solve, the interface and docs give the impression that it is supported for all algorithms.

using Optimization, OptimizationBBO

J(x,p) = x

F = OptimizationFunction(J)
prob = Optimization.OptimizationProblem(F, 0.0; lb = -10, ub =10, sense = MaxSense)
solve(prob, BBO_adaptive_de_rand_1_bin_radiuslimited()).objective # -10.0

Contributor Guide