Updated IV check to explicitly use len

CITCOM-project · Nov 27, 2024 · 75efc80 · 75efc80
1 parent f5400ee
commit 75efc80
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/causal_testing/specification/causal_dag.py b/causal_testing/specification/causal_dag.py
@@ -173,8 +173,8 @@ def check_iv_assumptions(self, treatment, outcome, instrument) -> bool:
             (
                 cause
                 for cause in self.graph.nodes
-                if list(nx.all_simple_paths(self.graph, source=cause, target=instrument))
-                and list(nx.all_simple_paths(self.graph, source=cause, target=outcome))
+                if len(list(nx.all_simple_paths(self.graph, source=cause, target=instrument))) > 0
+                and len(list(nx.all_simple_paths(self.graph, source=cause, target=outcome))) > 0
             )
         ):
             raise ValueError(f"Instrument {instrument} and outcome {outcome} share common causes")