Skip to content
This repository has been archived by the owner on Jan 9, 2024. It is now read-only.

Commit

Permalink
Removed DropMixin
Browse files Browse the repository at this point in the history
  • Loading branch information
cchoquette committed Jul 25, 2019
1 parent de2b0ff commit 0971f4c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions foreshadow/cleaners/internals/drop.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
import pandas as pd

from foreshadow.cleaners.data_cleaner import BaseCleaner
from foreshadow.core.preparerstep import DropMixin
from foreshadow.utils.validation import check_df


Expand All @@ -29,7 +28,7 @@ def drop_transform(text):
return text, res


class DropCleaner(BaseCleaner, DropMixin):
class DropCleaner(BaseCleaner):
"""Clean financial data.
Note: requires pandas input dataframes.
Expand Down
2 changes: 1 addition & 1 deletion foreshadow/transformers/core/notransform.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
"""No Transform class through acts as a pass through for DataFrame and flag."""
from sklearn.base import BaseEstimator, TransformerMixin

from foreshadow.transformers.core import make_pandas_transformer
from foreshadow.transformers.core.wrapper import make_pandas_transformer


@make_pandas_transformer
Expand Down

0 comments on commit 0971f4c

Please sign in to comment.