Merge pull request #364 from till-m/constraints-logging

bwheelz36 · web-flow · commit 8e5c89f65a7f · 2022-10-11T07:09:05.000+11:00
Fix JSONLogger with constraints
diff --git a/bayes_opt/logger.py b/bayes_opt/logger.py
@@ -128,6 +128,9 @@ def update(self, event, instance):
                 "delta": time_delta,
             }
 
+            if "allowed" in data: # fix: github.com/fmfn/BayesianOptimization/issues/361
+                data["allowed"] = bool(data["allowed"])
+
             with open(self._path, "a") as f:
                 f.write(json.dumps(data) + "\n")
 

Original file line number	Diff line number	Diff line change
`@@ -128,6 +128,9 @@ def update(self, event, instance):`
`128`	`128`	`"delta": time_delta,`
`129`	`129`	`}`
`130`	`130`
	`131`	`+ if "allowed" in data: # fix: github.com/fmfn/BayesianOptimization/issues/361`
	`132`	`+ data["allowed"] = bool(data["allowed"])`
	`133`	`+`
`131`	`134`	`with open(self._path, "a") as f:`
`132`	`135`	`f.write(json.dumps(data) + "\n")`
`133`	`136`