From e45891ab493c82e6bf1d8b59a791831525a34b83 Mon Sep 17 00:00:00 2001 From: Alyssa Travitz Date: Wed, 17 Jun 2026 10:17:16 -0700 Subject: [PATCH 1/2] transformations - use parent class init --- src/gufe/transformations/transformation.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/gufe/transformations/transformation.py b/src/gufe/transformations/transformation.py index adba1dd4..a7293020 100644 --- a/src/gufe/transformations/transformation.py +++ b/src/gufe/transformations/transformation.py @@ -213,10 +213,9 @@ def __init__( self._stateA = stateA self._stateB = stateB - self._protocol = protocol self._mapping = mapping - self._name = name - self._metadata = metadata or {} + + super().__init__(protocol=protocol, name=name, metadata=metadata) if validate: self.protocol.validate( @@ -313,9 +312,7 @@ def __init__( """ self._system = system - self._protocol = protocol - self._name = name - self._metadata = metadata or {} + super().__init__(protocol=protocol, name=name, metadata=metadata) if validate: self.protocol.validate( From a493bb16a3d7d1fd6621b772c08cb7d1debb7b62 Mon Sep 17 00:00:00 2001 From: Alyssa Travitz Date: Mon, 22 Jun 2026 14:26:42 -0700 Subject: [PATCH 2/2] move super inits to top --- src/gufe/transformations/transformation.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gufe/transformations/transformation.py b/src/gufe/transformations/transformation.py index a7293020..087397b4 100644 --- a/src/gufe/transformations/transformation.py +++ b/src/gufe/transformations/transformation.py @@ -204,6 +204,8 @@ def __init__( :meth:`.Protocol.validate` """ + super().__init__(protocol=protocol, name=name, metadata=metadata) + if isinstance(mapping, dict): warnings.warn( ("mapping input as a dict is deprecated; instead use either a single Mapping or list"), @@ -215,8 +217,6 @@ def __init__( self._stateB = stateB self._mapping = mapping - super().__init__(protocol=protocol, name=name, metadata=metadata) - if validate: self.protocol.validate( stateA=self.stateA, @@ -310,9 +310,9 @@ def __init__( Whether or not to validate the inputs to be provided to the :class:`.Protocol`. """ + super().__init__(protocol=protocol, name=name, metadata=metadata) self._system = system - super().__init__(protocol=protocol, name=name, metadata=metadata) if validate: self.protocol.validate(