Skip to content

BUG: Fix for extraneous default cell format in xlsxwriter files. #9171

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

Closed
wants to merge 1 commit into from
Closed

BUG: Fix for extraneous default cell format in xlsxwriter files. #9171

wants to merge 1 commit into from

Conversation

jmcnamara
Copy link
Contributor

Fix for issue in the xlsxwriter engine where is adds a default
'General' format to cells if no other format is applied. This
isn't a bug, per se, but it prevents other row or column formatting.

closes #9167

_skip_if_no_xlsxwriter()

import warnings
with warnings.catch_warnings():
Copy link
Contributor

Choose a reason for hiding this comment

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

_skip_if_no_openpyxl() doesn't suffice here?

@jreback jreback added Bug IO Excel read_excel, to_excel labels Jan 2, 2015
@jreback jreback added this to the 0.16.0 milestone Jan 2, 2015
@jreback
Copy link
Contributor

jreback commented Jan 2, 2015

@jmcnamara pls add a release note. This is a bug fix? user noticeable? (so an API change?) pls make the appropriate entry. thxs.

@jmcnamara
Copy link
Contributor Author

add a release note. This is a bug fix? user noticeable? (so an API change?) pls make the appropriate entry. thxs.

I'm not sure if it needs a release note entry since it doesn't really affect pandas. It it mainly to allow users to format columns in the encapsulated xlsxwriter object. If you think that it needs an entry I'll add one. Should it go in doc/source/whatsnew/v0.16.0.txt?

@jreback
Copy link
Contributor

jreback commented Jan 3, 2015

it needs a note - in v0.16.0 bug fix section

Fix for issue in the xlsxwriter engine where is adds a default
'General' format to cells if no other format is applied. This
isn't a bug, per se, but it prevents other row or column formatting.

Closes #9167
@jmcnamara
Copy link
Contributor Author

@jreback

  • Added release note.
  • Used _skip_if_no_openpyxl() as noted above.
  • Rebased and re-pushed.

@jreback
Copy link
Contributor

jreback commented Jan 6, 2015

merged via 07a5735

thanks!

@jreback jreback closed this Jan 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug IO Excel read_excel, to_excel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: ExcelWriter with xlsxwriter adds default format to cells preventing column formats
2 participants