Merge pull request biolab#3466 from VesnaT/fix_apply_selection

[FIX] OWDataProjectionWidget: Fix applying selection

Orange/widgets/visualize/tests/test_owscatterplot.py

@@ -282,9 +282,10 @@ def test_invalid_points_selection(self):
             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)
+        data = self.data.copy()[:11]
+        data[0, 0] = np.nan
+        self.send_signal(self.widget.Inputs.data, data)
+        self.assertIsNone(self.get_output(self.widget.Outputs.selected_data))
 
     def test_set_strings_settings(self):
         """

Orange/widgets/visualize/utils/widget.py

@@ -490,11 +490,10 @@ def setup_plot(self):
 
     # Selection
     def apply_selection(self):
-        if self.data is not None and self.__pending_selection is not None \
-                and self.graph.n_valid:
-            index_group = [(index, group) for index, group in
-                           self.__pending_selection if index < len(self.data)]
-            index_group = np.array(index_group).T
+        pending = self.__pending_selection
+        if self.data is not None and pending is not None and len(pending) \
+                and max(i for i, _ in pending) < self.graph.n_valid:
+            index_group = np.array(pending).T
             selection = np.zeros(self.graph.n_valid, dtype=np.uint8)
             selection[index_group[0]] = index_group[1]