PCA - Making lint happier

Author: PrimozGodec

Orange/widgets/unsupervised/owpca.py

@@ -5,6 +5,7 @@
 from AnyQt.QtCore import Qt
 
 from orangewidget.report import bool_str
+from orangewidget.settings import Setting
 
 from Orange.data import Table, Domain, StringVariable, ContinuousVariable
 from Orange.data.util import get_unique_names

Orange/widgets/unsupervised/tests/test_owpca.py

@@ -4,15 +4,15 @@
 from unittest.mock import patch, Mock
 
 import numpy as np
+from sklearn.utils import check_random_state
+from sklearn.utils.extmath import svd_flip
 
 from Orange.data import Table, Domain, ContinuousVariable, TimeVariable
 from Orange.preprocess import preprocess
 from Orange.widgets.tests.base import WidgetTest
 from Orange.widgets.tests.utils import table_dense_sparse, possible_duplicate_table
 from Orange.widgets.unsupervised.owpca import OWPCA
 from Orange.tests import test_filename
-from sklearn.utils import check_random_state
-from sklearn.utils.extmath import svd_flip
 
 
 class TestOWPCA(WidgetTest):
@@ -63,19 +63,19 @@ def test_limit_components(self):
             self.widget._setup_plot()  # pylint: disable=protected-access
 
     def test_migrate_settings_limits_components(self):
-        settings = dict(ncomponents=10)
+        settings = {"ncomponents": 10}
         OWPCA.migrate_settings(settings, 0)
         self.assertEqual(settings['ncomponents'], 10)
-        settings = dict(ncomponents=101)
+        settings = {"ncomponents": 101}
         OWPCA.migrate_settings(settings, 0)
         self.assertEqual(settings['ncomponents'], 100)
 
     def test_migrate_settings_changes_variance_covered_to_int(self):
-        settings = dict(variance_covered=17.5)
+        settings = {"variance_covered": 17.5}
         OWPCA.migrate_settings(settings, 0)
         self.assertEqual(settings["variance_covered"], 17)
 
-        settings = dict(variance_covered=float('nan'))
+        settings = {"variance_covered": float('nan')}
         OWPCA.migrate_settings(settings, 0)
         self.assertEqual(settings["variance_covered"], 100)
 
@@ -282,7 +282,7 @@ def test_table_subclass(self):
         When input table is instance of Table's subclass (e.g. Corpus) resulting
         tables should also be an instance subclasses
         """
-        class TableSub(Table):
+        class TableSub(Table):  # pylint: disable=abstract-method
             pass
 
         table_subclass = TableSub(self.iris)