Skip to content

Commit

Permalink
Merge pull request #211 from jhlegarreta/RemoveRedundantOverride
Browse files Browse the repository at this point in the history
ENH: Remove redundant `is_fitted` overrides in model classes
  • Loading branch information
oesteban committed Jun 13, 2024
2 parents 8c0bf36 + 5e4c301 commit dd38275
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions src/eddymotion/model/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,6 @@ def __init__(self, mask=None, **kwargs):
# Keep model state
self._model = None # "Main" model
self._models = None # For parallel (chunked) execution
self._is_fitted = False

# Setup brain mask
self._mask = mask
Expand Down Expand Up @@ -457,7 +456,6 @@ def __init__(self, **kwargs):
self._bias = kwargs.get("bias", True)
self._stat = kwargs.get("stat", "median")
self._data = None
self._is_fitted = False

def fit(self, data, **kwargs):
"""Calculate the average."""
Expand Down Expand Up @@ -487,10 +485,6 @@ def fit(self, data, **kwargs):
self._data = avg_func(shells, axis=-1)
self._is_fitted = True

@property
def is_fitted(self):
return self._is_fitted

def predict(self, *_, **kwargs):
"""Return the average map."""

Expand Down

0 comments on commit dd38275

Please sign in to comment.