Fix mutable default argument in __deepcopy__ methods

merlion/models/base.py

@@ -103,7 +103,9 @@ def __reduce__(self):
     def __copy__(self):
         return self.from_dict(self.to_dict())
 
-    def __deepcopy__(self, memodict={}):
+    def __deepcopy__(self, memodict=None):
+        if memodict is None:
+            memodict = {}
         return self.__copy__()
 
     def get_unused_kwargs(self, **kwargs):
@@ -467,7 +469,9 @@ def __copy__(self):
         new_model.__setstate__(state_dict)
         return new_model
 
-    def __deepcopy__(self, memodict={}):
+    def __deepcopy__(self, memodict=None):
+        if memodict is None:
+            memodict = {}
         new_model = self.__class__(config=copy.deepcopy(self.config))
         state_dict = self.__getstate__()
         state_dict.pop("config", None)

merlion/models/ensemble/base.py

@@ -71,7 +71,9 @@ def __copy__(self):
         config_dict["models"] = self.models
         return self.from_dict(config_dict)
 
-    def __deepcopy__(self, memodict={}):
+    def __deepcopy__(self, memodict=None):
+        if memodict is None:
+            memodict = {}
         copied = copy.copy(self)
         copied.models = copy.deepcopy(self.models)
         return copied

merlion/models/ensemble/combine.py

@@ -83,7 +83,9 @@ def from_dict(cls, state):
     def __copy__(self):
         return self.from_dict(self.to_dict())
 
-    def __deepcopy__(self, memodict={}):
+    def __deepcopy__(self, memodict=None):
+        if memodict is None:
+            memodict = {}
         return self.__copy__()
 
     @abstractmethod

merlion/models/layers.py

@@ -129,7 +129,9 @@ def __copy__(self):
         config_dict["model"] = self.model
         return self.from_dict(config_dict)
 
-    def __deepcopy__(self, memodict={}):
+    def __deepcopy__(self, memodict=None):
+        if memodict is None:
+            memodict = {}
         config_dict = super().to_dict(_skipped_keys={"model"})
         config_dict["model"] = copy.deepcopy(self.model)
         return self.from_dict(config_dict)

merlion/post_process/base.py

@@ -38,7 +38,9 @@ def from_dict(cls, state_dict):
     def __copy__(self):
         return self.from_dict(self.to_dict())
 
-    def __deepcopy__(self, memodict={}):
+    def __deepcopy__(self, memodict=None):
+        if memodict is None:
+            memodict = {}
         return self.__copy__()
 
     def __repr__(self):

merlion/utils/conj_priors.py

@@ -88,7 +88,9 @@ def __copy__(self):
             setattr(ret, k, copy.deepcopy(v))
         return ret
 
-    def __deepcopy__(self, memodict={}):
+    def __deepcopy__(self, memodict=None):
+        if memodict is None:
+            memodict = {}
         return self.__copy__()
 
     @staticmethod