Skip to content

Periods not correctly represented in to_excel() with monthly PeriodIndex #4133

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.

Sign up for GitHub

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

Jump to bottom
Closed
andreas-h opened this issue Jul 5, 2013 · 4 comments
Closed

Periods not correctly represented in to_excel() with monthly PeriodIndex #4133

andreas-h opened this issue Jul 5, 2013 · 4 comments
Labels
Enhancement IO Data IO issues that don't fit into a more specific label IO Excel read_excel, to_excel
Milestone
0.13.1

Comments

@andreas-h
Copy link
Contributor

When I have a Panel with a PeriodIndex with freq="M" and I use mypanel.to_excel(), then the PeriodIndex gets represented as YYYY-MM-DD in the output Excel file. I would expect it to be rather YYYY-MM.

Is this a bug or a feature?
@jreback
Copy link
Contributor

jreback commented Jul 5, 2013

not implemented

to_excel should deal with different kinds of indices better

welcome to have a PR though!

@jmcnamara
Copy link
Contributor

We could add ExcelWriter options to pass default date and time formats as strings. Currently the date/time format strings are hardcoded.

This would be relatively easy to implement. Feel free to assign the issue/PR to me.

@jtratner
Copy link
Contributor

@jmcnamara you are assigned to this :)

@jmcnamara jmcnamara mentioned this issue Nov 28, 2013
Closed
@jreback
Copy link
Contributor

jreback commented Jan 26, 2014

closed via 19ae1dc

@jreback jreback closed this as completed Jan 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement IO Data IO issues that don't fit into a more specific label IO Excel read_excel, to_excel
Projects
None yet
Milestone
0.13.1
Development

Successfully merging a pull request may close this issue.

ENH: Excel writer takes locale setting for date and datetime into account Jaydyou/pandas
4 participants
@jmcnamara @andreas-h @jreback @jtratner