You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The current setup.py fixes the openpyxl dependency at 3.0.9 because later versions break one very specific segment of code in adapter.to_pythonhere.
I think this could be fixed with:
if hasattr(self.wb.defined_names,"definedName"):
# This case is for openpyxl <3.1.0
all_input_ranges = {object_range.name for object_range in self.wb.defined_names.definedName}
else:
# This case is for openpyxl ≥3.1.0
all_input_ranges = set(self.wb.defined_names.keys())
This change was capable of getting around the error in my local environment and working, but it should be tried and ran through the adapter tests. If all seems good, then I'd advise just getting rid of the version requirement on openpyxl
The text was updated successfully, but these errors were encountered:
The current
setup.py
fixes the openpyxl dependency at3.0.9
because later versions break one very specific segment of code inadapter.to_python
here.I think this could be fixed with:
This change was capable of getting around the error in my local environment and working, but it should be tried and ran through the adapter tests. If all seems good, then I'd advise just getting rid of the version requirement on openpyxl
The text was updated successfully, but these errors were encountered: