Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add .xlsx export method #1241

Open
devinbalkind opened this issue Mar 6, 2016 · 2 comments
Open

Add .xlsx export method #1241

devinbalkind opened this issue Mar 6, 2016 · 2 comments

Comments

@devinbalkind
Copy link
Member

Minimally we should catch cases where this fails & report nicely to the user.
Better solutions are:
(1) Use .xlsx instead (requries using http://xlsxwriter.readthedocs.org/en/latest/index.html instead of xlwt)
(2) Export a multi-sheet workbook
Ideally all 3 are done:
.xls requests give a multi-sheet workbook with a user-warning
.xlsx requests are possible
The simple warning is the 'easy' part
Multi-sheet workbook seems fairly simple.
xlsx export is not really 'hard' but not 'easy'

@devinbalkind
Copy link
Member Author

anurag029 said:

I have tried to implement the multi-sheet workbook export using xlwt - anurag-ks@926c0d3
Need a review on it.

@nursix nursix self-assigned this Jun 27, 2016
@nursix
Copy link
Member

nursix commented Jun 27, 2016

Anaurag's multi-sheet export is now in trunk, which prevents the crash. The implementation of the xslx export options is still desirable, however - even though such massive exports are rather rare (and super-size spreadsheets of questionable value).

Downgrading to minor enhancement.

@nursix nursix removed their assignment Jun 27, 2016
@flavour flavour changed the title [trac import 10/11/14 by flavour] XLS exports limited to 65536 rows Add .xlsx export method Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants