Remove deprecated substation_nodes

Signed-off-by: Thijs Baaijen <[email protected]>

Author: Thijss

src/power_grid_model_ds/_core/model/graphs/models/base.py

@@ -2,7 +2,6 @@
 #
 # SPDX-License-Identifier: MPL-2.0
 
-import warnings
 from abc import ABC, abstractmethod
 from contextlib import contextmanager
 from typing import TYPE_CHECKING, Generator
@@ -253,23 +252,16 @@ def get_all_paths(self, ext_start_node_id: int, ext_end_node_id: int) -> list[li
 
         return [self._internals_to_externals(path) for path in internal_paths]
 
-    def get_components(self, substation_nodes: list[int] | None = None) -> list[list[int]]:
-        """Returns all separate components when the substation_nodes are removed of the graph as lists"""
-        if substation_nodes:
-            warnings.warn(
-                message="""
-get_components: substation_nodes argument is deprecated and will be removed in a future release.
-The functionality is still available with the use of the `tmp_remove_nodes` context manager.
-
-Example:
->>> with graph.tmp_remove_nodes(substation_nodes):
->>>    components = graph.get_components()
-""",
-                category=DeprecationWarning,
-                stacklevel=2,
-            )
-        with self.tmp_remove_nodes(substation_nodes or []):
-            internal_components = self._get_components()
+    def get_components(self) -> list[list[int]]:
+        """Returns all separate components when the substation_nodes are removed of the graph as lists
+
+        If you want to get the components of the graph without certain nodes, use the `tmp_remove_nodes` context manager.
+
+        Example:
+        >>> with graph.tmp_remove_nodes(substation_nodes):
+        >>>    components = graph.get_components()
+        """
+        internal_components = self._get_components()
         return [self._internals_to_externals(component) for component in internal_components]
 
     def get_connected(

tests/unit/model/graphs/test_graph_model.py

@@ -192,13 +192,14 @@ def test_get_components(graph_with_2_routes: BaseGraphModel):
     assert set(components[2]) == {100}
 
 
-def test_get_components_with_substation_nodes(graph_with_2_routes):
+def test_get_components_with_tmp_removed_substation_nodes(graph_with_2_routes):
     graph = graph_with_2_routes
     graph.add_node(99)
     graph.add_branch(1, 99)
     substation_nodes = np.array([1])
 
-    components = graph.get_components(substation_nodes=substation_nodes)
+    with graph.tmp_remove_nodes(substation_nodes):
+        components = graph.get_components()
 
     assert len(components) == 3
     assert set(components[0]) == {2, 3}