Update mlpack_optimizer.cpp - cmaes lower and upper were swapped

eclipse · Aug 25, 2023 · 61e130b · 61e130b
1 parent d1edaa7
commit 61e130b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/xacc/optimizer/mlpack/mlpack_optimizer.cpp b/xacc/optimizer/mlpack/mlpack_optimizer.cpp
@@ -192,7 +192,7 @@ OptResult MLPACKOptimizer::optimize(OptFunction &function) {
     if (options.keyExists<double>("mlpack-cmaes-lower-bound")) {
       lower = options.get<double>("mlpack-cmaes-lower-bound");
     }
-    CMAES<> optimizer(lambda, upper, lower, 1, maxiter, tol);
+    CMAES<> optimizer(lambda, lower, upper, 1, maxiter, tol);
     results = optimizer.Optimize(f, coordinates);
 #else
     xacc::error("Cannot run mlpack cmaes algorithm, lapack not found.");
@@ -240,4 +240,4 @@ OptResult MLPACKOptimizer::optimize(OptFunction &function) {
 
 } // namespace xacc
 
-REGISTER_OPTIMIZER(xacc::MLPACKOptimizer)
+REGISTER_OPTIMIZER(xacc::MLPACKOptimizer)