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

remove PeriodIndex.tz_convert, tz_localize? #21781

Closed
jbrockmendel opened this issue Jul 7, 2018 · 0 comments

Comments

Projects
None yet
2 participants
@jbrockmendel
Copy link
Member

commented Jul 7, 2018

ATM they just raise NotImplementedError("Not yet implemented for PeriodIndex"); if the idea of Period ever gaining a tz is out, these should be too.

@jreback jreback added this to the Contributions Welcome milestone Jul 7, 2018

@jreback jreback modified the milestones: Contributions Welcome, 0.24.0 Jul 16, 2018

gfyoung added a commit that referenced this issue Jul 18, 2018

CLN: Remove PeriodIndex.tz_convert, tz_localize (#21935)
CLN: Removed tz_convert and tz_localize from PeriodIndex

Closes gh-21781

aeltanawy pushed a commit to aeltanawy/pandas that referenced this issue Jul 20, 2018

CLN: Remove PeriodIndex.tz_convert, tz_localize (pandas-dev#21935)
CLN: Removed tz_convert and tz_localize from PeriodIndex

Closes pandas-devgh-21781

alimcmaster1 added a commit to alimcmaster1/pandas that referenced this issue Aug 12, 2018

CLN: Remove PeriodIndex.tz_convert, tz_localize (pandas-dev#21935)
CLN: Removed tz_convert and tz_localize from PeriodIndex

Closes pandas-devgh-21781

Sup3rGeo added a commit to Sup3rGeo/pandas that referenced this issue Oct 1, 2018

CLN: Remove PeriodIndex.tz_convert, tz_localize (pandas-dev#21935)
CLN: Removed tz_convert and tz_localize from PeriodIndex

Closes pandas-devgh-21781
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.