Skip to content

BUG: df.reset_index loses tz #7533

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

Merged
merged 1 commit into from
Jun 21, 2014
Merged

BUG: df.reset_index loses tz #7533

merged 1 commit into from
Jun 21, 2014

Conversation

sinhrks
Copy link
Member

@sinhrks sinhrks commented Jun 21, 2014

Closes #3950.

@@ -11876,6 +11876,22 @@ def test_reset_index_with_datetimeindex_cols(self):
datetime(2013, 1, 2)])
assert_frame_equal(result, expected)

def test_reset_index_with_tz(self):
# GH 3950
Copy link
Contributor

Choose a reason for hiding this comment

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

their are already some tests for for this issue
can u locate these near them?

Copy link
Contributor

Choose a reason for hiding this comment

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

put them here (either inside this function, or right after), just so that they are together: https://github.com/pydata/pandas/blob/master/pandas/tests/test_frame.py#L2137

@jreback jreback added this to the 0.14.1 milestone Jun 21, 2014
@sinhrks
Copy link
Member Author

sinhrks commented Jun 21, 2014

OK, modified locs.

@jreback
Copy link
Contributor

jreback commented Jun 21, 2014

needs a rebase

jreback added a commit that referenced this pull request Jun 21, 2014
BUG: df.reset_index loses tz
@jreback jreback merged commit cc452b2 into pandas-dev:master Jun 21, 2014
@jreback
Copy link
Contributor

jreback commented Jun 21, 2014

thanks!

@sinhrks sinhrks deleted the resetindex branch June 21, 2014 22:29
@jreback
Copy link
Contributor

jreback commented Jun 22, 2014

Don't use range(5) in building test frames, instead use np.arange(5,dtype='int64') to avoid dtype test issues on windows!

thanks

1a6d483

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tz info lost when creating multiindex
2 participants