Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
PERF: fast inf checking in to_excel #11352
Conversation
jorisvandenbossche
added Performance IO Excel
labels
Oct 17, 2015
jorisvandenbossche
added this to the
0.17.1
milestone
Oct 17, 2015
|
Looks good to me! |
|
I'd rather make a bigger change to how the xls writing is done, to make it much more performant. The idea is to copy what is done for csv writing. You chunk write, then convert the columns by blocks with Then you simply iterate and yield the |
|
Yeah, I had actually started looking into that - this particular change was low hanging fruit, so I thought it might make sense to do first. There probably is some performance to be picked up still, but an awful lot of the time is spent in the actual writing steps now.
|
|
@chris-b1 agreed. Ok rebase this (prob a whatsnew conflict). The can look at other improvements if desired. |
|
@jreback - rebased |
jreback
added a commit
that referenced
this pull request
Oct 17, 2015
|
|
jreback |
2431425
|
chris-b1 commentedOct 17, 2015
Adds new functions to check for infinity rather than calling
np.isposinfandnp.isneginf, which were (surprising to me) a significant drag onto_excel.I also added
xlsxwriterto the asv build configuration.openpyxlis still failing, I'm assuming something do with the specific version on conda?