[CI] Workflow for automated release

.github/workflows/release.yml

@@ -0,0 +1,12 @@
+name: Release
+on:
+  release:
+    types: [published]
+  workflow_dispatch:
+
+jobs:
+  release:
+    uses: biolab/orange-ci-cd/.github/workflows/release.yml@master
+    with:
+      pure-python: true
+    secrets: inherit

orangecontrib/single_cell/tests/test_load_data.py

@@ -76,12 +76,13 @@ def test_file_summary_gz(self):
         self.assertEqual(round(loader.sparsity, 2), 0.86)
 
     def test_file_summary_xls(self):
-        file_name = os.path.join(os.path.dirname(__file__), "data/data.xlsx")
-        loader = ExcelLoader(file_name)
-        self.assertEqual(loader.file_size, 9160)
-        self.assertEqual(loader.n_rows, 11)
-        self.assertEqual(loader.n_cols, 15)
-        self.assertEqual(round(loader.sparsity, 2), 0.86)
+        for file, size in (("data/data.xlsx", 9160), ("data/data.xls", 27648)):
+            file_name = os.path.join(os.path.dirname(__file__), file)
+            loader = ExcelLoader(file_name)
+            self.assertEqual(loader.file_size, size)
+            self.assertEqual(loader.n_rows, 11)
+            self.assertEqual(loader.n_cols, 15)
+            self.assertEqual(round(loader.sparsity, 2), 0.86)
 
     def test_file_summary_loom(self):
         file_name = os.path.join(os.path.dirname(__file__), "data/data.loom")
@@ -121,17 +122,18 @@ def test_load_data_mtx(self):
         npt.assert_array_equal(X, array)
 
     def test_load_data_xls(self):
-        kwargs = {"header_rows": 0, "header_cols": 0}
-        xls_path = os.path.join(os.path.dirname(__file__), "data/data.xlsx")
-        xls_loader = ExcelLoader(xls_path)
-        xls_attrs, xls_X, xls_M, xls_M_index = xls_loader._load_data(**kwargs)
-        csv_loader = Loader(os.path.join(os.path.dirname(__file__),
-                                         "data/DATA_MATRIX_LOG_TPM.txt"))
-        csv_attrs, csv_X, csv_M, csv_M_index = csv_loader._load_data(**kwargs)
-        self.assertEqual(xls_attrs, csv_attrs)
-        npt.assert_array_almost_equal(xls_X, csv_X)
-        npt.assert_array_equal(xls_M, csv_M)
-        npt.assert_array_equal(xls_M_index, csv_M_index)
+        for file in ("data/data.xlsx", "data/data.xls"):
+            kwargs = {"header_rows": 0, "header_cols": 0}
+            xls_path = os.path.join(os.path.dirname(__file__), file)
+            xls_loader = ExcelLoader(xls_path)
+            xls_attrs, xls_X, xls_M, xls_M_index = xls_loader._load_data(**kwargs)
+            csv_loader = Loader(os.path.join(os.path.dirname(__file__),
+                                             "data/DATA_MATRIX_LOG_TPM.txt"))
+            csv_attrs, csv_X, csv_M, csv_M_index = csv_loader._load_data(**kwargs)
+            self.assertEqual(xls_attrs, csv_attrs)
+            npt.assert_array_almost_equal(xls_X, csv_X)
+            npt.assert_array_equal(xls_M, csv_M)
+            npt.assert_array_equal(xls_M_index, csv_M_index)
 
     def test_n_genes_n_cells(self):
         file_name = os.path.join(os.path.dirname(__file__),
@@ -200,3 +202,5 @@ def test_concatenate_union(self):
         self.assertEqual(2 * len(data1) + len(data2), len(concat_data))
         self.assertEqual(len(concat_data.domain.attributes), 8)
         self.assertEqual(len(concat_data.domain.metas), 2)
+
+

orangecontrib/single_cell/widgets/load_data.py

@@ -18,6 +18,7 @@
     guess_data_type, sanitize_variable
 )
 from Orange.widgets.utils.filedialogs import RecentPath
+from openpyxl.reader.excel import load_workbook
 
 
 def separator_from_filename(file_name):
@@ -616,9 +617,16 @@ def __init__(self, file_name):
 
     def _set_file_parameters(self):
         try:
-            sheet = xlrd.open_workbook(self._file_name).sheet_by_index(0)
-            self.n_cols = sheet.ncols
-            self.n_rows = sheet.nrows
+            if self._file_name.endswith(".xls"):
+                # xlrd support only historic xls files
+                sheet = xlrd.open_workbook(self._file_name).sheet_by_index(0)
+                self.n_cols, self.n_rows = sheet.ncols, sheet.nrows
+            elif self._file_name.endswith(".xlsx"):
+                # use openpyxl library for xlsx files
+                wb = load_workbook(self._file_name, read_only=True)
+                sheet = wb.worksheets[0]
+                self.n_cols, self.n_rows = sheet.max_column, sheet.max_row
+                wb.close()
             self._set_sparsity()
         except Exception:
             pass

setup.py

@@ -15,7 +15,8 @@
             'fastdtw==0.3.2',
             'pandas>=0.23',
             'loompy>=2.0.10',
-            'xlrd~=1.2.0',
+            'xlrd>=2.0.1',
+            'openpyxl',
             'anndata>=0.6.21',
             'numpy',
             'scikit-learn',