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

ENH: Make ExcelWriter & ExcelFile contextmanagers #4933

Merged

Conversation

jtratner
Copy link
Contributor

Fixes #3441.

both can be used in with statements now. Makes it easier to use with multiple
writers, etc.

Plus touch up the docs on writers and such.

engine to use by default via the options ``io.excel.xlsx.writer`` and
By default, ``pandas`` uses ``openpyxl`` for ``.xlsx`` and ``.xlsm`` files and
``xlwt`` for ``.xls`` files. If you have multiple engines installed, you can
customize default engine to use via the options ``io.excel.xlsx.writer`` and
``io.excel.xls.writer``.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe show an example?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how about we just link to the section on config options? Might keep things simpler.

@jreback
Copy link
Contributor

jreback commented Sep 22, 2013

nice!

both can be used in with statements now. Makes it easier to use with
multiple writers, etc.

Plus touch up the docs on writers and such.
@jtratner
Copy link
Contributor Author

@cpcloud docs for this look okay?

@cpcloud
Copy link
Member

cpcloud commented Sep 23, 2013

👍

jtratner added a commit that referenced this pull request Sep 23, 2013
ENH: Make ExcelWriter & ExcelFile contextmanagers
@jtratner jtratner merged commit 30e2c6b into pandas-dev:master Sep 23, 2013
@jtratner jtratner deleted the make-ExcelWriter-contextmanager branch September 23, 2013 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow ExcelWriter() to add sheets to existing workbook
3 participants