Merge pull request #40 from causy-dev/remove-no-action

replace do_nothing actions on graph with empty returns
causy-dev · Jun 18, 2024 · 749e0c8 · 749e0c8
2 parents 042b6ae + 478d2ba
commit 749e0c8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/causy/causal_discovery/constraint/orientation_rules/fci.py b/causy/causal_discovery/constraint/orientation_rules/fci.py
@@ -49,7 +49,7 @@ def process(
 
         # if u and v are adjacent, do nothing
         if graph.undirected_edge_exists(x, y):
-            return TestResult(u=x, v=y, action=TestResultAction.DO_NOTHING, data={})
+            return
 
         # if u and v are NOT adjacent, store all shared adjacent nodes
         potential_zs = set(graph.edges[x.id].keys()).intersection(

diff --git a/causy/causal_discovery/constraint/orientation_rules/pc.py b/causy/causal_discovery/constraint/orientation_rules/pc.py
@@ -47,7 +47,7 @@ def process(
 
         # if u and v are adjacent, do nothing
         if graph.undirected_edge_exists(x, y):
-            return TestResult(u=x, v=y, action=TestResultAction.DO_NOTHING, data={})
+            return
 
         # if u and v are NOT adjacent, store all shared adjacent nodes
         potential_zs = set(graph.edges[x.id].keys()).intersection(