Merge pull request #2470 from ales-erjavec/fixes/table-del-ids

[FIX] table: Update `ids` in `Table.__del__`

Author: astaric

Orange/data/table.py

@@ -819,6 +819,7 @@ def __delitem__(self, key):
         self.Y = np.delete(self._Y, key, axis=0)
         self.metas = np.delete(self.metas, key, axis=0)
         self.W = np.delete(self.W, key, axis=0)
+        self.ids = np.delete(self.ids, key, axis=0)
 
     def __len__(self):
         return self.X.shape[0]

Orange/tests/test_table.py

@@ -356,6 +356,13 @@ def test_del_slice_example(self):
             del d[:]
             self.assertEqual(len(d), 0)
 
+    def test_del_slice_ids(self):
+        # __del__ updates `ids` member
+        d = data.Table("test2")
+        ids = d.ids.copy()
+        del d[2:4]
+        np.testing.assert_array_equal(d.ids, np.delete(ids, slice(2, 4)))
+
     def test_set_slice_example(self):
         import warnings