Made Adapters Sliceable (#285)

* in test phase, print statements included but get item implemented, need to define set item, might be nice to list indecies of transforms in the default print

* finished slice implementation for set item, noticed also that element transform was not added to the list for things to imported in the adapter file so added that

* finished adapter slicing, i also made a test file but its not in the commits

* ran linter

* removed print statements

* indexing added to print statements for adapter

* made modifications in line with Lars comments and they passed the tests that I had previously written

* removed unecessary comments2

* add basic list methods to Adapter

---------

Co-authored-by: larskue <[email protected]>

Author: eodole

bayesflow/adapters/adapter.py

@@ -1,4 +1,4 @@
-from collections.abc import Callable, Sequence
+from collections.abc import Callable, MutableSequence, Sequence
 
 import numpy as np
 from keras.saving import (
@@ -26,17 +26,16 @@
     ToArray,
     Transform,
 )
-
 from .transforms.filter_transform import Predicate
 
 
 @serializable(package="bayesflow.adapters")
-class Adapter:
+class Adapter(MutableSequence[Transform]):
     def __init__(self, transforms: Sequence[Transform] | None = None):
         if transforms is None:
             transforms = []
 
-        self.transforms = transforms
+        self.transforms = list(transforms)
 
     @staticmethod
     def create_default(inference_variables: Sequence[str]) -> "Adapter":
@@ -77,12 +76,70 @@ def __call__(self, data: dict[str, any], *, inverse: bool = False, **kwargs) ->
         return self.forward(data, **kwargs)
 
     def __repr__(self):
-        return f"Adapter([{' -> '.join(map(repr, self.transforms))}])"
+        result = ""
+        for i, transform in enumerate(self):
+            result += f"{i}: {transform!r}"
+            if i != len(self) - 1:
+                result += " -> "
 
-    def add_transform(self, transform: Transform):
-        self.transforms.append(transform)
+        return f"Adapter([{result}])"
+
+    # list methods
+
+    def append(self, value: Transform) -> "Adapter":
+        self.transforms.append(value)
         return self
 
+    def __delitem__(self, key: int | slice):
+        del self.transforms[key]
+
+    def extend(self, values: Sequence[Transform]) -> "Adapter":
+        if isinstance(values, Adapter):
+            values = values.transforms
+
+        self.transforms.extend(values)
+
+        return self
+
+    def __getitem__(self, item: int | slice) -> "Adapter":
+        if isinstance(item, int):
+            return self.transforms[item]
+
+        return Adapter(self.transforms[item])
+
+    def insert(self, index: int, value: Transform | Sequence[Transform]) -> "Adapter":
+        if isinstance(value, Adapter):
+            value = value.transforms
+
+        if isinstance(value, Sequence):
+            # convenience: Adapters are always flat
+            self.transforms = self.transforms[:index] + list(value) + self.transforms[index:]
+        else:
+            self.transforms.insert(index, value)
+
+        return self
+
+    def __setitem__(self, key: int | slice, value: Transform | Sequence[Transform]) -> "Adapter":
+        if isinstance(value, Adapter):
+            value = value.transforms
+
+        if isinstance(key, int) and isinstance(value, Sequence):
+            if key < 0:
+                key += len(self.transforms)
+
+            key = slice(key, key + 1)
+
+        self.transforms[key] = value
+
+        return self
+
+    def __len__(self):
+        return len(self.transforms)
+
+    # adapter methods
+
+    add_transform = append
+
     def apply(
         self,
         *,