-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
PERF: fast inf checking in to_excel #11352
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
Conversation
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. |
3a6eaf8
to
c5c44e8
Compare
@jreback - rebased |
PERF: fast inf checking in to_excel
Adds new functions to check for infinity rather than calling
np.isposinf
andnp.isneginf
, which were (surprising to me) a significant drag onto_excel
.I also added
xlsxwriter
to the asv build configuration.openpyxl
is still failing, I'm assuming something do with the specific version on conda?