removed positionmix entirely (now also in collection)

marl_factory_grid/environment/groups/agents.py

@@ -1,12 +1,21 @@
 from marl_factory_grid.environment.entity.agent import Agent
 from marl_factory_grid.environment.groups.collection import Collection
-from marl_factory_grid.environment.groups.mixins import PositionMixin
 
 
-class Agents(PositionMixin, Collection):
+class Agents(Collection):
     _entity = Agent
-    is_blocking_light = False
-    can_move = True
+
+    @property
+    def var_is_blocking_light(self):
+        return False
+
+    @property
+    def var_can_move(self):
+        return True
+
+    @property
+    def var_has_position(self):
+        return True
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)

marl_factory_grid/environment/groups/collection.py

@@ -1,9 +1,12 @@
+from typing import List, Tuple
+
+from marl_factory_grid.environment.entity.entity import Entity
 from marl_factory_grid.environment.groups.objects import _Objects
 from marl_factory_grid.environment.entity.object import EnvObject
 
 
 class Collection(_Objects):
-    _entity = EnvObject
+    _entity = EnvObject  # entity? object? objects?
 
     @property
     def var_is_blocking_light(self):
@@ -20,11 +23,12 @@ class Collection(_Objects):
     @property
     def var_has_position(self):
         return False  # alles was posmixin hat true
+
     @property
     def var_has_bound(self):
         return False  # batteries, globalpos, inventories true
 
-    @property   # beide bounds hier? inventory can be bound
+    @property  # beide bounds hier? inventory can be bound
     def var_can_be_bound(self):
         return False
 
@@ -62,3 +66,44 @@ class Collection(_Objects):
             return next((idx for idx, x in enumerate(self) if x.belongs_to_entity(entity)))
         except (StopIteration, AttributeError):
             return None
+
+    def spawn(self, coords: List[Tuple[(int, int)]]):
+        self.add_items([self._entity(pos) for pos in coords])
+
+    def render(self):
+        return [y for y in [x.render() for x in self] if y is not None]
+
+    @classmethod
+    def from_coordinates(cls, positions: [(int, int)], *args, entity_kwargs=None, **kwargs, ):
+        collection = cls(*args, **kwargs)
+        collection.add_items(
+            [cls._entity(tuple(pos), **entity_kwargs if entity_kwargs is not None else {}) for pos in positions])
+        return collection
+
+    def __delitem__(self, name):
+        idx, obj = next((i, obj) for i, obj in enumerate(self) if obj.name == name)
+        try:
+            for observer in obj.observers:
+                observer.notify_del_entity(obj)
+        except AttributeError:
+            pass
+        super().__delitem__(name)
+
+    def by_pos(self, pos: (int, int)):
+        pos = tuple(pos)
+        try:
+            return self.pos_dict[pos]
+        except StopIteration:
+            pass
+        except ValueError:
+            print()
+
+    @property
+    def positions(self):
+        return [e.pos for e in self]
+
+    def notify_del_entity(self, entity: Entity):
+        try:
+            self.pos_dict[entity.pos].remove(entity)
+        except (ValueError, AttributeError):
+            pass

marl_factory_grid/environment/groups/mixins.py

@@ -1,55 +1,4 @@
-from typing import List, Tuple
-
 from marl_factory_grid.environment import constants as c
-from marl_factory_grid.environment.entity.entity import Entity
-
-
-class PositionMixin:
-    _entity = Entity
-    var_is_blocking_light: bool = True
-    var_can_collide: bool = True
-    var_has_position: bool = True
-
-    def spawn(self, coords: List[Tuple[(int, int)]]):
-        self.add_items([self._entity(pos) for pos in coords])
-
-    def render(self):
-        return [y for y in [x.render() for x in self] if y is not None]
-
-    @classmethod
-    def from_coordinates(cls, positions: [(int, int)], *args, entity_kwargs=None, **kwargs, ):
-        collection = cls(*args, **kwargs)
-        collection.add_items(
-            [cls._entity(tuple(pos), **entity_kwargs if entity_kwargs is not None else {}) for pos in positions])
-        return collection
-
-    def __delitem__(self, name):
-        idx, obj = next((i, obj) for i, obj in enumerate(self) if obj.name == name)
-        try:
-            for observer in obj.observers:
-                observer.notify_del_entity(obj)
-        except AttributeError:
-            pass
-        super().__delitem__(name)
-
-    def by_pos(self, pos: (int, int)):
-        pos = tuple(pos)
-        try:
-            return self.pos_dict[pos]
-        except StopIteration:
-            pass
-        except ValueError:
-            print()
-
-    @property
-    def positions(self):
-        return [e.pos for e in self]
-
-    def notify_del_entity(self, entity: Entity):
-        try:
-            self.pos_dict[entity.pos].remove(entity)
-        except (ValueError, AttributeError):
-            pass
 
 
 # noinspection PyUnresolvedReferences,PyTypeChecker

marl_factory_grid/environment/groups/utils.py

@@ -1,17 +1,14 @@
 from typing import List, Union
 
-import numpy as np
-
 from marl_factory_grid.environment.entity.util import GlobalPosition
 from marl_factory_grid.environment.groups.collection import Collection
-from marl_factory_grid.environment.groups.mixins import PositionMixin, HasBoundMixin
-from marl_factory_grid.environment.groups.objects import _Objects
-from marl_factory_grid.modules.zones import Zone
-from marl_factory_grid.utils import helpers as h
-from marl_factory_grid.environment import constants as c
 
 
-class Combined(PositionMixin, Collection):
+class Combined(Collection):
+
+    @property
+    def var_has_position(self):
+        return True
 
     @property
     def name(self):
@@ -38,8 +35,14 @@ class Combined(PositionMixin, Collection):
 class GlobalPositions(Collection):
 
     _entity = GlobalPosition
-    is_blocking_light = False,
-    can_collide = False
+
+    @property
+    def var_is_blocking_light(self):
+        return False
+
+    @property
+    def var_can_collide(self):
+        return False
 
     def __init__(self, *args, **kwargs):
         super(GlobalPositions, self).__init__(*args, **kwargs)

marl_factory_grid/environment/groups/walls.py

@@ -1,13 +1,16 @@
 from marl_factory_grid.environment import constants as c
 from marl_factory_grid.environment.entity.wall import Wall
 from marl_factory_grid.environment.groups.collection import Collection
-from marl_factory_grid.environment.groups.mixins import PositionMixin
 
 
-class Walls(PositionMixin, Collection):
+class Walls(Collection):
     _entity = Wall
     symbol = c.SYMBOL_WALL
 
+    @property
+    def var_has_position(self):
+        return True
+
     def __init__(self, *args, **kwargs):
         super(Walls, self).__init__(*args, **kwargs)
         self._value = c.VALUE_OCCUPIED_CELL