NiaPy icon indicating copy to clipboard operation
NiaPy copied to clipboard

xlwt is archived: consider dropping xlwt requirement?

Open sanjayankur31 opened this issue 4 years ago • 6 comments

Is your feature request related to a problem? Please describe. The xlwt has been archived, and will no longer receive updates. https://github.com/python-excel/xlwt

See also: https://groups.google.com/g/python-excel/c/P6TjJgFVjMI/m/g8d0eWxTBQAJ

Upstream suggests that one should not use these utils, and instead use openpyxl.

Describe the solution you'd like Drop xlwt based functionality, or migrate to another still maintained library.

Describe alternatives you've considered At the moment none, we're using the latest 1.3.0 version that is needed by NiaPy, but if there are bugs there it'll be hard to fix them now with an archived xlwt.

Additional context NA

sanjayankur31 avatar Dec 04 '20 09:12 sanjayankur31

Thank you @sanjayankur31

This issue has already been resolved. However, xlwt will stay on dep list in order to ensure backward compatibility.

firefly-cpp avatar Dec 05 '20 14:12 firefly-cpp

Hi @firefly-cpp, openpyxl still is a hard dependency or now is an optional dependency for NiaPy?

carlosal1015 avatar Dec 20 '23 03:12 carlosal1015

Hi @carlosal1015, currently openpyxl is still a hard dependency. Maybe we can work something out in the future releases. If you have any propositions on how to elegantly solve this, you are more than welcome to create a PR.

GregaVrbancic avatar Dec 20 '23 06:12 GregaVrbancic

Currently, this is still a hard dependency. However, we may solve it without using external libraries. @zStupan, do you think there are other ways to eliminate this dependency? It is practically used only for dumping the results into an Excel file.

firefly-cpp avatar Dec 20 '23 08:12 firefly-cpp

@firefly-cpp We could make it an extra, so it would only be installed if the user requests it, like pip install niapy[excel] or something.

zStupan avatar Dec 20 '23 09:12 zStupan

Good idea.

firefly-cpp avatar Dec 20 '23 10:12 firefly-cpp