Removed unused depends_on_outputs

Author: jmafoster1

causal_testing/specification/causal_dag.py

@@ -489,22 +489,6 @@ def get_backdoor_graph(self, treatments: list[str]) -> CausalDAG:
         backdoor_graph.add_edges_from(filter(lambda x: x not in outgoing_edges, self.edges))
         return backdoor_graph
 
-    def depends_on_outputs(self, node: Node, scenario: Scenario) -> bool:
-        """Check whether a given node in a given scenario is or depends on a
-        model output in the given scenario. That is, whether or not the model
-        needs to be run to determine its value.
-
-        NOTE: The graph must be acyclic for this to terminate.
-
-        :param node: The node in the DAG representing the variable of interest.
-        :param scenario: The modelling scenario.
-
-        :return: Whether the given variable is or depends on an output.
-        """
-        if isinstance(scenario.variables[node], Output):
-            return True
-        return any((self.depends_on_outputs(n, scenario) for n in self.predecessors(node)))
-
     @staticmethod
     def remove_hidden_adjustment_sets(minimal_adjustment_sets: list[str], scenario: Scenario):
         """Remove variables labelled as hidden from adjustment set(s)

tests/specification_tests/test_causal_dag.py

@@ -362,58 +362,6 @@ def tearDown(self) -> None:
         shutil.rmtree(self.temp_dir_path)
 
 
-class TestDependsOnOutputs(unittest.TestCase):
-    """
-    Test the depends_on_outputs method.
-    """
-
-    def setUp(self) -> None:
-        from scipy.stats import uniform
-        from causal_testing.specification.variable import Input, Output, Meta
-        from causal_testing.specification.scenario import Scenario
-
-        self.temp_dir_path = tempfile.mkdtemp()
-        self.dag_dot_path = os.path.join(self.temp_dir_path, "dag.dot")
-        dag_dot = """digraph G { A -> B; B -> C; D -> A; D -> C}"""
-        f = open(self.dag_dot_path, "w")
-        f.write(dag_dot)
-        f.close()
-
-        D = Input("D", float, uniform(0, 1))
-        A = Meta("A", float, uniform(0, 1))
-        B = Output("B", float, uniform(0, 1))
-        C = Meta("C", float, uniform(0, 1))
-
-        self.scenario = Scenario({A, B, C, D})
-
-    def test_depends_on_outputs_output(self):
-        causal_dag = CausalDAG(self.dag_dot_path)
-        print("nodes:", causal_dag.nodes())
-        print("graph:", causal_dag)
-        self.assertTrue(causal_dag.depends_on_outputs("B", self.scenario))
-
-    def test_depends_on_outputs_output_meta(self):
-        causal_dag = CausalDAG(self.dag_dot_path)
-        print("nodes:", causal_dag.nodes())
-        print("graph:", causal_dag)
-        self.assertTrue(causal_dag.depends_on_outputs("C", self.scenario))
-
-    def test_depends_on_outputs_input_meta(self):
-        causal_dag = CausalDAG(self.dag_dot_path)
-        print("nodes:", causal_dag.nodes())
-        print("graph:", causal_dag)
-        self.assertFalse(causal_dag.depends_on_outputs("A", self.scenario))
-
-    def test_depends_on_outputs_input(self):
-        causal_dag = CausalDAG(self.dag_dot_path)
-        print("nodes:", causal_dag.nodes())
-        print("graph:", causal_dag)
-        self.assertFalse(causal_dag.depends_on_outputs("D", self.scenario))
-
-    def tearDown(self) -> None:
-        shutil.rmtree(self.temp_dir_path)
-
-
 class TestUndirectedGraphAlgorithms(unittest.TestCase):
     """
     Test the graph algorithms designed for the undirected graph variants of a Causal DAG.