Change from class to free function accepting parameter struct.

doc/roots/differential_evolution.qbk

@@ -14,73 +14,68 @@ LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)
 
     namespace boost::math::tools {
 
-    template <typename BoundsContainer>
-    class differential_evolution {
-    public:
-        using BoundType = BoundsContainer::value_type;
-        using Real = BoundType::value_type;
-
-        differential_evolution(BoundsContainer bounds, Real F = 0.65, double crossover_ratio = 0.5, 
-                               size_t NP = 200, size_t max_generations = 1000, 
-                               size_t threads = std::thread::hardware_concurrency());
-
-        template <typename ArgumentContainer, class Func, class URBG>
-        ArgumentContainer argmin(const Func cost_function, URBG &g,
-                                  ArgumentContainer* initial_guess = nullptr,
-                                  std::invoke_result_t<Func, ArgumentContainer> target_value = 
-                                      std::numeric_limits<std::invoke_result_t<Func, ArgumentContainer>>::quiet_NaN(),
-                                  std::atomic<bool> *cancellation = nullptr,
-                                  std::vector<std::pair<ArgumentContainer, std::invoke_result_t<Func, ArgumentContainer>>>
-                                      *queries = nullptr,
-                                  std::atomic<std::invoke_result_t<Func, ArgumentContainer>>* current_minimum_cost = nullptr);
+    template <typename ArgumentContainer> struct differential_evolution_parameters {
+        using Real = typename ArgumentContainer::value_type;
+        ArgumentContainer lower_bounds;
+        ArgumentContainer upper_bounds;
+        Real F = static_cast<Real>(0.65);
+        double crossover_ratio = 0.5;
+        // Population in each generation:
+        size_t NP = 200;
+        size_t max_generations = 1000;
+        size_t threads = std::thread::hardware_concurrency();
+	ArgumentContainer const * initial_guess = nullptr;
     };
 
+    template <typename ArgumentContainer, class Func, class URBG>
+    ArgumentContainer differential_evolution(
+        const Func cost_function, differential_evolution_parameters<ArgumentContainer> const &de_params, URBG &g,
+        std::invoke_result_t<Func, ArgumentContainer> target_value = std::numeric_limits<std::invoke_result_t<Func, ArgumentContainer>>::quiet_NaN(),
+        std::atomic<bool> *cancellation = nullptr,
+        std::vector<std::pair<ArgumentContainer, std::invoke_result_t<Func, ArgumentContainer>>> *queries = nullptr,
+        std::atomic<std::invoke_result_t<Func, ArgumentContainer>> *current_minimum_cost = nullptr);
+
     } // namespaces
 ``
 
-The `differential_evolution` class provides an implementation of the (classical) differential evolution optimization algorithm, often going by the label `de/rand/bin/1`.
+The `differential_evolution` function provides an implementation of the (classical) differential evolution optimization algorithm, often going by the label `de/rand/bin/1`.
 It is capable of minimizing a cost function defined on a continuous space represented by a set of bounds.
 
-[heading Constructor]
 
-``
-differential_evolution(BoundsContainer bounds, Real F = 0.8, double crossover_ratio = 0.5, 
-                       size_t NP = 1000, size_t max_generations = 1000, size_t threads = std::thread::hardware_concurrency());
-``
-
-Parameters:
+[heading Parameters]
 
-    `bounds`: A container representing the bounds of the optimization space. The `.size()` of the bounds should return the dimension of the problem, and each element of the `bounds` should have two elements of the same type.
+    `lower_bounds`: A container representing the lower bounds of the optimization space along each dimension. The `.size()` of the bounds should return the dimension of the problem.
+    `upper_bounds`: A container representing the upper bounds of the optimization space along each dimension. It should have the same size of `lower_bounds`, and each element should be >= the corresponding element of `lower_bounds`.
     `F`: The scale factor controlling the rate at which the population evolves (default is 0.65).
     `crossover_ratio`: The crossover ratio determining the trade-off between exploration and exploitation (default is 0.5).
     `NP`: The population size (default is 200). Parallelization occurs over the population, so this should be "large".
     `max_generations`: The maximum number of generations for the optimization (default is 1000).
     `threads`: The number of threads to use for parallelization (default is the hardware concurrency). If the objective function is already multithreaded, then this should be set to 1 to prevent oversubscription.
+    `initial_guess`: An optional guess for where we should start looking for solutions.
 
 The defaults were chosen by a reading of Price, Storn, and Lampinen, chapter 3, with the exception of the population size, which we have chosen a bit larger than they found as core counts have increased since publication of this reference and parallelization occurs within each generation.
-There is a tradeoff between finding global minima and convergence speed.
+Note that there is a tradeoff between finding global minima and convergence speed.
 The most robust way of decreasing the probability of getting stuck in a local minima is to increase the population size.
 
-[heading Member Function]
+[heading The function]
 
 ``
 template <typename ArgumentContainer, class Func, class URBG>
-ArgumentContainer argmin(const Func cost_function, URBG &rng,
-                         ArgumentContainer* initial_guess = nullptr,
-                         std::invoke_result_t<Func, ArgumentContainer> target_value = 
-                              std::numeric_limits<std::invoke_result_t<Func, ArgumentContainer>>::quiet_NaN(),
-                         std::atomic<bool> *cancellation = nullptr,
-                         std::vector<std::pair<ArgumentContainer, std::invoke_result_t<Func, ArgumentContainer>>>
-                              *queries = nullptr,
-                         std::atomic<std::invoke_result_t<Func, ArgumentContainer>>* current_minimum_cost = nullptr);
+ArgumentContainer differential_evolution(const Func cost_function,
+    differential_evolution_parameters<ArgumentContainer> const &de_params,
+    URBG &g,
+    std::invoke_result_t<Func, ArgumentContainer> value_to_reach = std::numeric_limits<std::invoke_result_t<Func, ArgumentContainer>>::quiet_NaN(),
+    std::atomic<bool> *cancellation = nullptr,
+    std::vector<std::pair<ArgumentContainer, std::invoke_result_t<Func, ArgumentContainer>>> *queries = nullptr,
+    std::atomic<std::invoke_result_t<Func, ArgumentContainer>> *current_minimum_cost = nullptr)
 ``
 
 Parameters:
 
     `cost_function`: The cost function to be minimized.
+    `de_params`: The parameters to the algorithm as described above.
     `rng`: A uniform random bit generator, like `std::mt19937_64`.
-    `initial_guess`: An optional initial guess for the optimal solution.
-    `value_to_reach`: An optional value that, if reached, stops the optimization. This is the most robust way to terminate the calculation, but in most cases the optimal value of the cost function is unknown, but if it is, use it! See the referenced book for clear examples of when target values can be deduced. 
+    `value_to_reach`: An optional value that, if reached, stops the optimization. This is the most robust way to terminate the calculation, but in most cases the optimal value of the cost function is unknown. If it is, use it! See the referenced book for clear examples of when target values can be deduced. 
     `cancellation`: An optional atomic boolean to allow the user to stop the computation and gracefully return the best result found up to that point. N.B.: Cancellation is not immediate; the in-progress generation finishes.
     `queries`: An optional vector to store intermediate results during optimization. This is useful for debugging and perhaps volume rendering of the objective function after the calculation is complete.
     `current_minimum_cost`: An optional atomic variable to store the current minimum cost during optimization. This allows developers to (e.g.) plot the progress of the minimization over time and in conjunction with the `cancellation` argument allow halting the computation when the progress stagnates. Refer to Price, Storn, and Lampinen, Section 3.2 for caveats with this approach.