Update factory/world_factory.py

factory/world_factory.py

@@ -42,6 +42,7 @@ from __future__ import annotations
 import copy
 import inspect
 import logging
+import numpy as np
 from collections.abc import Mapping, MutableMapping, Sequence
 from dataclasses import dataclass, field
 from importlib import import_module
@@ -374,6 +375,35 @@ class WorldFactory:
         owner = owner_agent_id or "world"
         index = 0 if behavior_index is None else int(behavior_index)
         return f"{owner}:behavior:{index}:{spec.name}"
+    
+    @staticmethod
+    def _agent_runtime_position(position: Any) -> np.ndarray:
+        """Return a NumPy position vector compatible with core.agent.Agent.
+
+        The current Agent implementation expects position to behave like a NumPy
+        array. In particular, Agent.observe() and Agent.snapshot() use array-style
+        methods such as copy() and tolist().
+        """
+
+        if position is None:
+            return np.zeros(2, dtype=np.float64)
+
+        try:
+            array = np.asarray(position, dtype=np.float64).reshape(-1)
+        except (TypeError, ValueError) as exc:
+            raise RuntimeObjectBuildError(
+                f"Agent position must be numeric, got {position!r}"
+            ) from exc
+
+        if array.size == 0:
+            return np.zeros(2, dtype=np.float64)
+
+        if not np.all(np.isfinite(array)):
+            raise RuntimeObjectBuildError(
+                f"Agent position must contain only finite values, got {position!r}"
+            )
+
+        return array
 
     def create_world(self, spec: WorldSpec | Mapping[str, Any]) -> World:
         """Create a runtime world from a validated or mappable world spec.
@@ -459,10 +489,12 @@ class WorldFactory:
         )
         policy = self.create_policy_for_agent(spec)
 
+        runtime_position = self._agent_runtime_position(spec.position)
+
         params = {
             "id": spec.id,
             "type": spec.type,
-            "position": None if spec.position is None else tuple(spec.position),
+            "position": runtime_position,
             "state": copy.deepcopy(spec.state),
             "memory": copy.deepcopy(spec.memory),
             "goals": copy.deepcopy(spec.goals),
@@ -482,7 +514,7 @@ class WorldFactory:
 
         self._safe_setattr(agent, "id", spec.id)
         self._safe_setattr(agent, "type", spec.type)
-        self._safe_setattr(agent, "position", None if spec.position is None else tuple(spec.position))
+        self._safe_setattr(agent, "position", runtime_position.copy())
         self._safe_setattr(agent, "state", copy.deepcopy(spec.state))
         self._safe_setattr(agent, "memory", copy.deepcopy(spec.memory))
         self._safe_setattr(agent, "goals", copy.deepcopy(spec.goals))