settings: Do not clear schema_only settings on close_context

* undo changes to settings from gh-2301
* change selection restore in scatter plot and linear projection accordingly

Orange/widgets/settings.py

@@ -787,17 +787,6 @@ def close_context(self, widget):
         if widget.current_context is None:
             return
 
-        # Clear schema-only settings when *closing* the context
-        # FIXME: Sadly, schema-only settings aren't cleared when non-contextual
-        # SettingsHandler is used. Example: widget has non-contextual
-        # SettingsHandler, user loads schema that includes widget, user
-        # passes new data to widget => widget's schema-only settings still
-        # have previous values because equivalent of below reset was never
-        # called (i.e. `close_context()`).
-        for name, setting in self.known_settings.items():
-            if setting.schema_only:
-                setattr(widget, name, setting.default)
-
         self.settings_from_widget(widget)
         widget.current_context = None
 

Orange/widgets/tests/test_context_handler.py

@@ -15,6 +15,7 @@ class SimpleWidget:
     settings_version = 1
 
     setting = Setting(42)
+    schema_only_setting = Setting(None, schema_only=True)
 
     context_setting = ContextSetting(42)
 
@@ -181,6 +182,17 @@ def test_write_defaults_stores_version(self):
             for c in contexts:
                 self.assertEqual(c.values.get("__version__", 0xBAD), 1)
 
+    def test_close_context(self):
+        handler = ContextHandler()
+        handler.bind(SimpleWidget)
+        widget = SimpleWidget()
+        widget.storeSpecificSettings = Mock()
+        handler.initialize(widget)
+        widget.schema_only_setting = 0xD06F00D
+        widget.current_context = handler.new_context()
+        handler.close_context(widget)
+        self.assertEqual(widget.schema_only_setting, 0xD06F00D)
+
 
 class TestSettingsPrinter(TestCase):
     def test_formats_contexts(self):

Orange/widgets/visualize/owlinearprojection.py

@@ -236,6 +236,8 @@ def __init__(self):
         self.__legend = None
         self.__selection_item = None
         self.__replot_requested = False
+        # Do we need to restore/apply saved state (selection)
+        self.__did_restore_state = self.selection_indices is None
 
         box = gui.vBox(self.controlArea, "Axes")
 
@@ -515,8 +517,10 @@ def set_data(self, data):
             if set(selected_keys).issubset(set(state.keys())):
                 pass
 
-            if self.selection_indices is not None:
+            if self.selection_indices is not None \
+                    and not self.__did_restore_state:
                 self.select_indices(self.selection_indices)
+                self.__did_restore_state = True
 
             # update the defaults state (the encoded state must contain
             # all variables in the input domain)
@@ -951,6 +955,7 @@ def commit(self):
             indices = numpy.flatnonzero(self._selection_mask)
             if len(indices) > 0:
                 subset = self.data[indices]
+            indices = indices.tolist()
 
         self.selection_indices = indices
         self.Outputs.selected_data.send(subset)

Orange/widgets/visualize/owscatterplot.py

@@ -144,6 +144,8 @@ def __init__(self):
         self.attribute_selection_list = None  # list of Orange.data.Variable
         self.__timer = QTimer(self, interval=1200)
         self.__timer.timeout.connect(self.add_data)
+        # Do we need to restore/apply saved state (selection)
+        self.__did_restore_state = self.selection_group is None
 
         common_options = dict(
             labelWidth=50, orientation=Qt.Horizontal, sendSelectedValue=True,
@@ -335,7 +337,8 @@ def handleNewSignals(self):
         self.update_graph()
         self.cb_class_density.setEnabled(self.graph.can_draw_density())
         self.cb_reg_line.setEnabled(self.graph.can_draw_regresssion_line())
-        self.apply_selection()
+        if not self.__did_restore_state:
+            self.apply_selection()
         self.unconditional_commit()
 
     def prepare_data(self):
@@ -376,6 +379,7 @@ def apply_selection(self):
             selection_array = np.array(self.selection_group).T
             self.graph.selection[selection_array[0]] = selection_array[1]
             self.graph.update_colors(keep_colors=True)
+            self.__did_restore_state = True
 
     @Inputs.features
     def set_shown_attributes(self, attributes):

Orange/widgets/visualize/tests/test_owscatterplot.py

@@ -234,7 +234,10 @@ def test_saving_selection(self):
 
     def test_points_selection(self):
         # Opening widget with saved selection should restore it
-        self.widget.selection_group = [(i, 1) for i in range(50)]
+        self.widget = self.create_widget(
+            OWScatterPlot, stored_settings={
+                "selection_group": [(i, 1) for i in range(50)]}
+        )
         self.send_signal(self.widget.Inputs.data, self.data)  # iris
         selected_data = self.get_output(self.widget.Outputs.selected_data)
         self.assertEqual(len(selected_data), 50)
@@ -253,7 +256,10 @@ def test_migrate_selection(self):
     def test_invalid_points_selection(self):
         # if selection contains rows that are not present in the current
         # dataset, widget should select what can be selected.
-        self.widget.selection_group = [(i, 1) for i in range(50)]
+        self.widget = self.create_widget(
+            OWScatterPlot, stored_settings={
+                "selection_group": [(i, 1) for i in range(50)]}
+        )
         self.send_signal(self.widget.Inputs.data, self.data[:10])
         selected_data = self.get_output(self.widget.Outputs.selected_data)
         self.assertEqual(len(selected_data), 10)