Merge pull request #241 from CITCOM-project/fix-json-adequacy-bug

Fixed "np.int64" is not JSON encodable from test outcomes

Author: jmafoster1

causal_testing/testing/causal_test_outcome.py

@@ -97,9 +97,9 @@ def apply(self, res: CausalTestResult) -> bool:
         if res.ci_valid() and not super().apply(res):
             return False
         if res.test_value.type in {"ate", "coefficient"}:
-            return res.test_value.value > 0
+            return bool(res.test_value.value > 0)
         if res.test_value.type == "risk_ratio":
-            return res.test_value.value > 1
+            return bool(res.test_value.value > 1)
         # Dead code but necessary for pylint
         raise ValueError(f"Test Value type {res.test_value.type} is not valid for this TestOutcome")
 
@@ -111,8 +111,8 @@ def apply(self, res: CausalTestResult) -> bool:
         if res.ci_valid() and not super().apply(res):
             return False
         if res.test_value.type in {"ate", "coefficient"}:
-            return res.test_value.value < 0
+            return bool(res.test_value.value < 0)
         if res.test_value.type == "risk_ratio":
-            return res.test_value.value < 1
+            return bool(res.test_value.value < 1)
         # Dead code but necessary for pylint
         raise ValueError(f"Test Value type {res.test_value.type} is not valid for this TestOutcome")