Fix bug where on_best did not receive iterate() kwargs (#63)

* Fix bug where on_best did not receive iterate() kwargs

* Bump version

alns/ALNS.py

@@ -176,7 +176,11 @@ class of vehicle routing problems with backhauls. *European Journal of
             destroyed = d_operator(curr, self._rnd_state, **kwargs)
             cand = r_operator(destroyed, self._rnd_state, **kwargs)
 
-            best, curr, s_idx = self._consider_candidate(crit, best, curr, cand)
+            best, curr, s_idx = self._eval_cand(crit,
+                                                best,
+                                                curr,
+                                                cand,
+                                                **kwargs)
 
             weight_scheme.update_weights(d_idx, r_idx, s_idx)
 
@@ -201,7 +205,7 @@ def on_best(self, func: _OperatorType):
         """
         self._on_best = func
 
-    def _consider_candidate(
+    def _eval_cand(
             self,
             crit: AcceptanceCriterion,
             best: State,

alns/tests/test_alns.py

@@ -192,6 +192,25 @@ def test_operator(state, rnd, item):
     alns.iterate(init_sol, weights, HillClimbing(), 10, item=orig_item)
 
 
+def test_bugfix_pass_kwargs_to_on_best():
+    """
+    Exercises a bug where the on_best callback did not receive the kwargs passed
+    to iterate().
+    """
+    def test_operator(state, rnd, item):
+        assert_(item is orig_item)
+        return Zero()  # better, so on_best is triggered
+
+    alns = get_alns_instance([lambda state, rnd, item: state], [test_operator])
+    alns.on_best(lambda state, rnd, item: state)
+
+    init_sol = One()
+    weights = SimpleWeights([1, 1, 1, 1], 1, 1, .5)
+    orig_item = object()
+
+    alns.iterate(init_sol, weights, HillClimbing(), 10, item=orig_item)
+
+
 # EXAMPLES ---------------------------------------------------------------------
 
 

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "alns"
-version = "2.1.1"
+version = "2.1.2"
 description = "A flexible implementation of the adaptive large neighbourhood search (ALNS) algorithm."
 authors = ["Niels Wouda <[email protected]>"]
 license = "MIT"