breaking: space: Remove deprecated position_agent

Author: rht

mesa/space.py

@@ -504,42 +504,6 @@ class SingleGrid(_Grid):
         torus: Boolean which determines whether to treat the grid as a torus.
     """
 
-    def position_agent(
-        self, agent: Agent, x: int | str = "random", y: int | str = "random"
-    ) -> None:
-        """Position an agent on the grid.
-        This is used when first placing agents! Setting either x or y to "random"
-        gives the same behavior as 'move_to_empty()' to get a random position.
-        If x or y are positive, they are used.
-        Use 'swap_pos()' to swap agents positions.
-        """
-        warn(
-            (
-                "`position_agent` is being deprecated; use instead "
-                "`place_agent` to place an agent at a specified "
-                "location or `move_to_empty` to place an agent "
-                "at a random empty cell."
-            ),
-            DeprecationWarning,
-        )
-
-        if not (isinstance(x, int) or x == "random"):
-            raise Exception(
-                "x must be an integer or a string 'random'."
-                f" Actual type: {type(x)}. Actual value: {x}."
-            )
-        if not (isinstance(y, int) or y == "random"):
-            raise Exception(
-                "y must be an integer or a string 'random'."
-                f" Actual type: {type(y)}. Actual value: {y}."
-            )
-
-        if x == "random" or y == "random":
-            self.move_to_empty(agent)
-        else:
-            coords = (x, y)
-            self.place_agent(agent, coords)
-
     def place_agent(self, agent: Agent, pos: Coordinate) -> None:
         """Place the agent at the specified location, and set its pos variable."""
         if self.is_cell_empty(pos):

tests/test_grid.py

@@ -268,25 +268,6 @@ def setUp(self):
                 self.grid.place_agent(a, (x, y))
         self.num_agents = len(self.agents)
 
-    @patch.object(MockAgent, "model", create=True)
-    def test_position_agent(self, mock_model):
-        a = MockAgent(100, None)
-        with self.assertRaises(Exception) as exc_info:
-            self.grid.position_agent(a, (1, 1))
-        expected = (
-            "x must be an integer or a string 'random'."
-            " Actual type: <class 'tuple'>. Actual value: (1, 1)."
-        )
-        assert str(exc_info.exception) == expected
-        with self.assertRaises(Exception) as exc_info:
-            self.grid.position_agent(a, "(1, 1)")
-        expected = (
-            "x must be an integer or a string 'random'."
-            " Actual type: <class 'str'>. Actual value: (1, 1)."
-        )
-        assert str(exc_info.exception) == expected
-        self.grid.position_agent(a, "random")
-
     @patch.object(MockAgent, "model", create=True)
     def test_enforcement(self, mock_model):
         """

tests/test_space.py

@@ -308,7 +308,7 @@ def test_empty_cells(self):
         if self.space.exists_empty_cells():
             for i, pos in enumerate(list(self.space.empties)):
                 a = MockAgent(-i, pos)
-                self.space.position_agent(a, x=pos[0], y=pos[1])
+                self.space.place_agent(a, x=pos[0], y=pos[1])
         with self.assertRaises(Exception):
             self.space.move_to_empty(a)