MNT deprecate passing kwargs by position (#721)

Author: glemaitre

doc/whats_new/v0.7.rst

@@ -19,6 +19,9 @@ Maintenance
 - Remove `FutureWarning` issued by `scikit-learn` 0.23.
   :pr:`710` by :user:`Guillaume Lemaitre <glemaitre>`.
 
+- Impose keywords only argument as in `scikit-learn`.
+  :pr:`721` by :user:`Guillaume Lemaitre <glemaitre>`.
+
 Changed models
 ..............
 
@@ -67,3 +70,7 @@ Deprecation
   :class:`imblearn.under_sampling.ClusterCentroids` since it was used by
   :class:`sklearn.cluster.KMeans` which deprecated it.
   :pr:`710` by :user:`Guillaume Lemaitre <glemaitre>`.
+
+- Deprecation of passing keyword argument by position similarly to
+  `scikit-learn`.
+  :pr:`721` by :user:`Guillaume lemaitre <glemaitre>`.

imblearn/base.py

@@ -14,6 +14,7 @@
 
 from .utils import check_sampling_strategy, check_target_type
 from .utils._validation import ArraysTransformer
+from .utils._validation import _deprecate_positional_args
 
 
 class SamplerMixin(BaseEstimator, metaclass=ABCMeta):
@@ -213,7 +214,8 @@ class FunctionSampler(BaseSampler):
 
     _sampling_type = "bypass"
 
-    def __init__(self, func=None, accept_sparse=True, kw_args=None,
+    @_deprecate_positional_args
+    def __init__(self, *, func=None, accept_sparse=True, kw_args=None,
                  validate=True):
         super().__init__()
         self.func = func

imblearn/combine/_smote_enn.py

@@ -15,6 +15,7 @@
 from ..utils import Substitution
 from ..utils._docstring import _n_jobs_docstring
 from ..utils._docstring import _random_state_docstring
+from ..utils._validation import _deprecate_positional_args
 
 
 @Substitution(
@@ -85,8 +86,10 @@ class SMOTEENN(BaseSampler):
 
     _sampling_type = "over-sampling"
 
+    @_deprecate_positional_args
     def __init__(
         self,
+        *,
         sampling_strategy="auto",
         random_state=None,
         smote=None,

imblearn/combine/_smote_tomek.py

@@ -16,6 +16,7 @@
 from ..utils import Substitution
 from ..utils._docstring import _n_jobs_docstring
 from ..utils._docstring import _random_state_docstring
+from ..utils._validation import _deprecate_positional_args
 
 
 @Substitution(
@@ -85,8 +86,10 @@ class SMOTETomek(BaseSampler):
 
     _sampling_type = "over-sampling"
 
+    @_deprecate_positional_args
     def __init__(
         self,
+        *,
         sampling_strategy="auto",
         random_state=None,
         smote=None,

imblearn/datasets/_imbalance.py

@@ -9,10 +9,12 @@
 
 from ..under_sampling import RandomUnderSampler
 from ..utils import check_sampling_strategy
+from ..utils._validation import _deprecate_positional_args
 
 
+@_deprecate_positional_args
 def make_imbalance(
-    X, y, sampling_strategy=None, random_state=None, verbose=False, **kwargs
+    X, y, *, sampling_strategy=None, random_state=None, verbose=False, **kwargs
 ):
     """Turns a dataset into an imbalanced dataset with a specific sampling
     strategy.

imblearn/datasets/_zenodo.py

@@ -57,6 +57,8 @@
 from sklearn.utils import Bunch
 from sklearn.utils import check_random_state
 
+from ..utils._validation import _deprecate_positional_args
+
 URL = (
     "https://zenodo.org/record/61452/files/"
     "benchmark-imbalanced-learn.tar.gz"
@@ -101,7 +103,9 @@
     MAP_ID_NAME[v + 1] = k
 
 
+@_deprecate_positional_args
 def fetch_datasets(
+    *,
     data_home=None,
     filter_data=None,
     download_if_missing=True,

imblearn/datasets/tests/test_imbalance.py

@@ -32,14 +32,14 @@ def test_make_imbalance_error(iris, sampling_strategy, err_msg):
     # cover in the common tests so we will repeat it here
     X, y = iris
     with pytest.raises(ValueError, match=err_msg):
-        make_imbalance(X, y, sampling_strategy)
+        make_imbalance(X, y, sampling_strategy=sampling_strategy)
 
 
 def test_make_imbalance_error_single_class(iris):
     X, y = iris
     y = np.zeros_like(y)
     with pytest.raises(ValueError, match="needs to have more than 1 class."):
-        make_imbalance(X, y, {0: 10})
+        make_imbalance(X, y, sampling_strategy={0: 10})
 
 
 @pytest.mark.parametrize(

imblearn/ensemble/_bagging.py

@@ -18,6 +18,7 @@
 from ..utils import Substitution, check_target_type, check_sampling_strategy
 from ..utils._docstring import _n_jobs_docstring
 from ..utils._docstring import _random_state_docstring
+from ..utils._validation import _deprecate_positional_args
 
 
 @Substitution(
@@ -175,10 +176,12 @@ class BalancedBaggingClassifier(BaggingClassifier):
      [  2 225]]
     """
 
+    @_deprecate_positional_args
     def __init__(
         self,
         base_estimator=None,
         n_estimators=10,
+        *,
         max_samples=1.0,
         max_features=1.0,
         bootstrap=True,

imblearn/ensemble/_easy_ensemble.py

@@ -17,6 +17,7 @@
 from ..utils import Substitution, check_target_type, check_sampling_strategy
 from ..utils._docstring import _n_jobs_docstring
 from ..utils._docstring import _random_state_docstring
+from ..utils._validation import _deprecate_positional_args
 from ..pipeline import Pipeline
 
 MAX_INT = np.iinfo(np.int32).max
@@ -125,10 +126,12 @@ class EasyEnsembleClassifier(BaggingClassifier):
      [  2 225]]
     """
 
+    @_deprecate_positional_args
     def __init__(
         self,
         n_estimators=10,
         base_estimator=None,
+        *,
         warm_start=False,
         sampling_strategy="auto",
         replacement=False,

imblearn/ensemble/_forest.py

@@ -34,6 +34,7 @@
 from ..utils._docstring import _n_jobs_docstring
 from ..utils._docstring import _random_state_docstring
 from ..utils._validation import check_sampling_strategy
+from ..utils._validation import _deprecate_positional_args
 
 MAX_INT = np.iinfo(np.int32).max
 
@@ -297,9 +298,11 @@ class labels (multi-output problem).
     [1]
     """
 
+    @_deprecate_positional_args
     def __init__(
         self,
         n_estimators=100,
+        *,
         criterion="gini",
         max_depth=None,
         min_samples_split=2,