RemoveNaNClasses: Deprecate and replace uses with HasClass

Author: lanzagar

Orange/base.py

@@ -8,10 +8,10 @@
 import scipy
 
 from Orange.data import Table, Storage, Instance, Value
+from Orange.data.filter import HasClass
 from Orange.data.util import one_hot
 from Orange.misc.wrapper_meta import WrapperMeta
-from Orange.preprocess import (RemoveNaNClasses, Continuize,
-                               RemoveNaNColumns, SklImpute, Normalize)
+from Orange.preprocess import Continuize, RemoveNaNColumns, SklImpute, Normalize
 from Orange.util import Reprable
 
 __all__ = ["Learner", "Model", "SklLearner", "SklModel"]
@@ -341,7 +341,7 @@ class SklLearner(Learner, metaclass=WrapperMeta):
     _params = {}
 
     preprocessors = default_preprocessors = [
-        RemoveNaNClasses(),
+        HasClass(),
         Continuize(),
         RemoveNaNColumns(),
         SklImpute()]

Orange/classification/rules.py

@@ -15,10 +15,11 @@
 import numpy as np
 from scipy.stats import chi2
 
-from Orange.data import Table, _contingency
 from Orange.classification import Learner, Model
+from Orange.data import Table, _contingency
+from Orange.data.filter import HasClass
 from Orange.preprocess.discretize import EntropyMDL
-from Orange.preprocess import RemoveNaNColumns, RemoveNaNClasses, Impute
+from Orange.preprocess import RemoveNaNColumns, Impute
 
 __all__ = ["CN2Learner", "CN2UnorderedLearner", "CN2SDLearner",
            "CN2SDUnorderedLearner"]
@@ -901,7 +902,7 @@ class _RuleLearner(Learner):
     .. [1] "Separate-and-Conquer Rule Learning", Johannes Fürnkranz,
            Artificial Intelligence Review 13, 3-54, 1999
     """
-    preprocessors = [RemoveNaNColumns(), RemoveNaNClasses(), Impute()]
+    preprocessors = [RemoveNaNColumns(), HasClass(), Impute()]
 
     def __init__(self, preprocessors=None, base_rules=None):
         """

Orange/classification/softmax_regression.py

@@ -2,8 +2,8 @@
 from scipy.optimize import fmin_l_bfgs_b
 
 from Orange.classification import Learner, Model
-from Orange.preprocess import (RemoveNaNClasses, Continuize, RemoveNaNColumns,
-                               Impute, Normalize)
+from Orange.data.filter import HasClass
+from Orange.preprocess import Continuize, RemoveNaNColumns, Impute, Normalize
 
 __all__ = ["SoftmaxRegressionLearner"]
 
@@ -40,7 +40,7 @@ class SoftmaxRegressionLearner(Learner):
         Parameters for L-BFGS algorithm.
     """
     name = 'softmax'
-    preprocessors = [RemoveNaNClasses(),
+    preprocessors = [HasClass(),
                      RemoveNaNColumns(),
                      Impute(),
                      Continuize(),

Orange/preprocess/preprocess.py

@@ -12,7 +12,7 @@
 from Orange.data.filter import HasClass
 from Orange.preprocess.util import _RefuseDataInConstructor
 from Orange.statistics import distribution
-from Orange.util import Reprable, Enum
+from Orange.util import Reprable, Enum, deprecated
 from . import impute, discretize, transformation
 
 __all__ = ["Continuize", "Discretize", "Impute",
@@ -198,6 +198,7 @@ def __call__(self, data):
         return data.transform(domain)
 
 
+@deprecated
 class RemoveNaNClasses(Preprocess):
     """
     Construct preprocessor that removes examples with missing class

Orange/preprocess/score.py

@@ -4,12 +4,13 @@
 
 import numpy as np
 from sklearn import feature_selection as skl_fss
-from Orange.misc.wrapper_meta import WrapperMeta
 
-from Orange.statistics import contingency, distribution
 from Orange.data import Domain, Variable, DiscreteVariable, ContinuousVariable
-from Orange.preprocess.preprocess import Discretize, Impute, RemoveNaNClasses
+from Orange.data.filter import HasClass
+from Orange.misc.wrapper_meta import WrapperMeta
+from Orange.preprocess.preprocess import Discretize, Impute
 from Orange.preprocess.util import _RefuseDataInConstructor
+from Orange.statistics import contingency, distribution
 from Orange.util import Reprable
 
 __all__ = ["Chi2",
@@ -27,9 +28,7 @@ class Scorer(_RefuseDataInConstructor, Reprable):
     feature_type = None
     class_type = None
     supports_sparse_data = None
-    preprocessors = [
-        RemoveNaNClasses()
-    ]
+    preprocessors = [HasClass]
 
     @property
     def friendly_name(self):

Orange/regression/linear_bfgs.py

@@ -1,9 +1,9 @@
 import numpy as np
 from scipy.optimize import fmin_l_bfgs_b
 
+from Orange.data.filter import HasClass
+from Orange.preprocess import Normalize, Continuize, Impute, RemoveNaNColumns
 from Orange.regression import Learner, Model
-from Orange.preprocess import (RemoveNaNClasses, Normalize, Continuize,
-                               Impute, RemoveNaNColumns)
 
 __all__ = ["LinearRegressionLearner"]
 
@@ -51,7 +51,7 @@ class LinearRegressionLearner(Learner):
         print(c(data)) # predict
     '''
     name = 'linear_bfgs'
-    preprocessors = [RemoveNaNClasses(),
+    preprocessors = [HasClass(),
                      Normalize(),
                      Continuize(),
                      Impute(),

Orange/tests/test_preprocess.py

@@ -78,6 +78,8 @@ def test_nothing_to_remove(self):
         self.assertEqual(len(d.domain.attributes), 4)
 
 
+# RemoveNaNClass has been deprecated in favor of HasClass
+# Tests will be removed together with the deprecated class
 class TestRemoveNanClass(unittest.TestCase):
     def test_remove_nan_classes(self):
         table = Table("imports-85")
@@ -122,7 +124,7 @@ def test_scaling_median_stddev(self):
 class TestReprs(unittest.TestCase):
     def test_reprs(self):
         preprocs = [Continuize, Discretize, Impute, SklImpute, Normalize,
-                    Randomize, RemoveNaNClasses, ProjectPCA, ProjectCUR, Scale,
+                    Randomize, ProjectPCA, ProjectCUR, Scale,
                     EqualFreq, EqualWidth, EntropyMDL, SelectBestFeatures,
                     SelectRandomFeatures, RemoveNaNColumns, DoNotImpute, DropInstances,
                     Average, Default]

Orange/tests/test_rules.py

@@ -3,18 +3,17 @@
 
 import unittest
 import numpy as np
-from Orange.data import Table
-from Orange.preprocess import RemoveNaNClasses, Impute
 
-from Orange.classification.rules import main as rules_main
+from Orange.classification import (CN2Learner, CN2UnorderedLearner,
+                                   CN2SDLearner, CN2SDUnorderedLearner)
 from Orange.classification.rules import (_RuleLearner, _RuleClassifier,
                                          RuleHunter, Rule, EntropyEvaluator,
                                          LaplaceAccuracyEvaluator,
                                          WeightedRelativeAccuracyEvaluator,
                                          argmaxrnd, hash_dist)
-
-from Orange.classification import (CN2Learner, CN2UnorderedLearner,
-                                   CN2SDLearner, CN2SDUnorderedLearner)
+from Orange.data import Table
+from Orange.data.filter import HasClass
+from Orange.preprocess import Impute
 
 
 class TestRuleInduction(unittest.TestCase):
@@ -41,7 +40,7 @@ def test_base_RuleLearner(self):
         self.assertEqual(len(list(base_rule_learner.active_preprocessors)), 3)
         # preprocessor types
         preprocessor_types = [type(x) for x in base_rule_learner.active_preprocessors]
-        self.assertIn(RemoveNaNClasses, preprocessor_types)
+        self.assertIn(HasClass, preprocessor_types)
         self.assertIn(Impute, preprocessor_types)
 
         # test find_rules

Orange/widgets/evaluate/owtestlearners.py

@@ -2,7 +2,6 @@
 # pylint: disable=invalid-sequence-index
 
 import sys
-import functools
 from itertools import chain
 import abc
 import enum
@@ -13,7 +12,6 @@
 
 import concurrent.futures
 from concurrent.futures import Future
-
 from collections import OrderedDict, namedtuple
 
 try:
@@ -31,21 +29,21 @@
 from AnyQt.QtCore import Qt, QSize, QThread, QMetaObject, Q_ARG
 from AnyQt.QtCore import pyqtSlot as Slot
 
+from Orange.base import Learner
+import Orange.classification
 from Orange.data import Table, DiscreteVariable, ContinuousVariable
+from Orange.data.filter import HasClass
 from Orange.data.sql.table import SqlTable, AUTO_DL_LIMIT
 import Orange.evaluation
-import Orange.classification
-import Orange.regression
-
-from Orange.base import Learner
 from Orange.evaluation import scoring, Results
 from Orange.preprocess.preprocess import Preprocess
-from Orange.preprocess import RemoveNaNClasses
+import Orange.regression
 from Orange.widgets import gui, settings, widget
 from Orange.widgets.utils.itemmodels import DomainModel
 from Orange.widgets.widget import OWWidget, Msg, Input, Output
 from Orange.widgets.utils.concurrent import ThreadExecutor
 
+
 log = logging.getLogger(__name__)
 
 InputLearner = namedtuple(
@@ -389,7 +387,7 @@ def set_train_data(self, data):
         if self.train_data_missing_vals or self.test_data_missing_vals:
             self.Warning.missing_data(self._which_missing_data())
             if data:
-                data = RemoveNaNClasses(data)
+                data = HasClass(data)
         else:
             self.Warning.missing_data.clear()
 
@@ -439,7 +437,7 @@ def set_test_data(self, data):
         if self.train_data_missing_vals or self.test_data_missing_vals:
             self.Warning.missing_data(self._which_missing_data())
             if data:
-                data = RemoveNaNClasses()(data)
+                data = HasClass()(data)
         else:
             self.Warning.missing_data.clear()