Merge pull request #6816 from ales-erjavec/revert-6737

[MNT] Revert #6737

Author: markotoplak

Orange/data/io.py

@@ -32,15 +32,6 @@
 from Orange.util import flatten
 
 
-try:
-    # temporary fix for file not closed issue until openpyxl prepare release
-    from Orange.misc.openpyxl_patch import read_worksheets
-
-    openpyxl.reader.excel.ExcelReader.read_worksheets = read_worksheets
-except:  # pylint: disable=bare-except
-    pass
-
-
 # Support values longer than 128K (i.e. text contents features)
 csv.field_size_limit(100*1024*1024)
 

Orange/misc/_distmatrix_xlsx.py

@@ -2,15 +2,6 @@
 import openpyxl
 
 
-try:
-    # temporary fix for file not closed issue until openpyxl prepare release
-    from Orange.misc.openpyxl_patch import read_worksheets
-
-    openpyxl.reader.excel.ExcelReader.read_worksheets = read_worksheets
-except:  # pylint: disable=bare-except
-    pass
-
-
 def read_matrix(filename, sheet_name=None):
     sheet = _get_sheet(filename, sheet_name)
     cells, empty_cols, empty_rows = _non_empty_cells(sheet)

Orange/misc/openpyxl_patch.py

@@ -56,7 +56,7 @@ requirements:
     - keyrings.alt
     - networkx
     - numpy >=1.20.0,<2
-    - openpyxl
+    - openpyxl >=3.1.3
     - openTSNE >=0.6.1,!=0.7.0
     - pandas >=1.4.0,!=1.5.0,!=2.0.0
     - packaging

Orange/misc/tests/test_openpyxl_patch.py

@@ -10,7 +10,7 @@ keyring
 keyrings.alt    # for alternative keyring implementations
 networkx
 numpy>=1.20.0,<2
-openpyxl
+openpyxl>=3.1.3
 openTSNE>=0.6.1,!=0.7.0  # 0.7.0 segfaults
 packaging
 pandas>=1.4.0,!=1.5.0,!=2.0.0

conda-recipe/meta.yaml

@@ -54,7 +54,7 @@ deps =
     # oldest: keyrings.alt
     # oldest: networkx
     oldest: numpy==1.20
-    # oldest: openpyxl
+    oldest: openpyxl==3.1.3
     oldest: openTSNE==0.6.1
     oldest: pandas==1.4.0
     oldest: pip==19.3