Merge pull request #351 from bwheelz36/fix_scipy_1.9

changed the way x_try is sent to minimize - fixes #350

Author: bwheelz36

bayes_opt/util.py

@@ -53,7 +53,7 @@ def acq_max(ac, gp, y_max, bounds, random_state, n_warmup=10000, n_iter=10):
     for x_try in x_seeds:
         # Find the minimum of minus the acquisition function
         res = minimize(lambda x: -ac(x.reshape(1, -1), gp=gp, y_max=y_max),
-                       x_try.reshape(1, -1),
+                       x_try,
                        bounds=bounds,
                        method="L-BFGS-B")
 

tests/test_bayesian_optimization.py

@@ -299,6 +299,27 @@ def test_define_wrong_transformer():
                                          bounds_transformer=3)
 
 
+def test_single_value_objective():
+    """
+    As documented [here](https://github.com/scipy/scipy/issues/16898)
+    scipy is changing the way they handle 1D objectives inside minimize.
+    This is a simple test to make sure our tests fail if scipy updates this
+    in future
+    """
+    pbounds = {'x': (-10, 10)}
+
+    optimizer = BayesianOptimization(
+        f=lambda x: x*3,
+        pbounds=pbounds,
+        verbose=2,
+        random_state=1,
+    )
+    optimizer.maximize(
+        init_points=2,
+        n_iter=3,
+    )
+
+
 if __name__ == '__main__':
     r"""
     CommandLine: