[FIX] ScatterPlot Crashes on Data With Infinity Values

Orange/widgets/tests/base.py

@@ -792,10 +792,10 @@ def missing_data_3(cls):
         return Table(cls.path("missing_data_3.tab"))
 
     @classmethod
-    def data_one_column_nans(cls):
+    def data_one_column_vals(cls, value=np.nan):
         """
         Data set with two continuous features and one discrete. One continuous
-        columns has missing values (NaN).
+        columns has custom set values (default nan).
 
         Returns
         -------
@@ -812,4 +812,21 @@ def data_one_column_nans(cls):
                 ["", "", "", ""],
                 "ynyn"
             )))
+        table[:, 1] = value
         return table
+
+    @classmethod
+    def data_one_column_nans(cls):
+        """
+        Data set with two continuous features and one discrete. One continuous
+        columns has missing values (NaN).
+
+        Returns
+        -------
+        data : Orange.data.Table
+        """
+        return cls.data_one_column_vals(value=np.nan)
+
+    @classmethod
+    def data_one_column_infs(cls):
+        return cls.data_one_column_vals(value=np.inf)

Orange/widgets/utils/scaling.py

@@ -49,7 +49,7 @@ def _compute_scaled_data(self):
         Y = data.Y if data.Y.ndim == 2 else np.atleast_2d(data.Y).T
         self.original_data = np.hstack((data.X, Y)).T
         self.scaled_data = no_jit = self.original_data.copy()
-        self.valid_data_array = ~np.isnan(no_jit)
+        self.valid_data_array = np.isfinite(no_jit)
         for index in range(len(data.domain)):
             attr = data.domain[index]
             if attr.is_discrete:

Orange/widgets/visualize/tests/test_owscatterplot.py

@@ -126,6 +126,17 @@ def test_data_column_nans(self):
 
         self.widget.update_graph()
 
+    def test_data_column_infs(self):
+        """
+        Scatter Plot should not crash on data with infinity values
+        GH-2707
+        GH-2684
+        """
+        table = datasets.data_one_column_infs()
+        self.send_signal(self.widget.Inputs.data, table)
+        attr_x = self.widget.controls.attr_x
+        simulate.combobox_activate_item(attr_x, "b")
+
     def test_regression_line(self):
         """It is possible to draw the line only for pair of continuous attrs"""
         self.send_signal(self.widget.Inputs.data, self.data)