Skip to content

DEPR: Deprecated PeriodIndex.to_datetime #14113

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

Conversation

gfyoung
Copy link
Member

@gfyoung gfyoung commented Aug 28, 2016

Continuation of #14096 by deprecating PeriodIndex.to_datetime in favour of PeriodIndex.to_timestamp.

Also addresses #8254.

@codecov-io
Copy link

codecov-io commented Aug 28, 2016

Current coverage is 85.27% (diff: 100%)

Merging #14113 into master will increase coverage by <.01%

@@             master     #14113   diff @@
==========================================
  Files           139        139          
  Lines         50511      50513     +2   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          43071      43074     +3   
+ Misses         7440       7439     -1   
  Partials          0          0          

Powered by Codecov. Last update 10bf721...9fd85bb

@jorisvandenbossche jorisvandenbossche added the Deprecate Functionality to remove in pandas label Aug 28, 2016
@gfyoung gfyoung force-pushed the index-to-datetime-depr-cont branch from 876d72c to 9fd85bb Compare August 30, 2016 01:15
@gfyoung
Copy link
Member Author

gfyoung commented Sep 1, 2016

@jreback , @jorisvandenbossche : Travis is passing. Ready to merge if there are no other concerns.

@jreback
Copy link
Contributor

jreback commented Sep 1, 2016

lgtm

@jorisvandenbossche jorisvandenbossche merged commit 306e647 into pandas-dev:master Sep 1, 2016
@jorisvandenbossche
Copy link
Member

Thanks!

@jorisvandenbossche jorisvandenbossche added this to the 0.19.0 milestone Sep 1, 2016
@gfyoung gfyoung deleted the index-to-datetime-depr-cont branch September 1, 2016 08:05
@gfyoung
Copy link
Member Author

gfyoung commented Sep 1, 2016

Don't forget to add to the list in #6581!

@jorisvandenbossche
Copy link
Member

@gfyoung Already did :-) (but linked to the issue, where the links are to all the different prs)

gfyoung added a commit to forking-repos/pandas that referenced this pull request Dec 5, 2017
Removes the following .to_datetime methods

* Index.to_datetime
* Timestamp.to_datetime
* PeriodIndex.to_datetime
* DatetimeIndex.to_datetime

All were deprecated in 0.19.0

xref pandas-devgh-8254, pandas-devgh-14096, pandas-devgh-14113
gfyoung added a commit to forking-repos/pandas that referenced this pull request Dec 6, 2017
Removes the following .to_datetime methods

* Index.to_datetime
* NaTType.to_datetime
* Timestamp.to_datetime
* PeriodIndex.to_datetime
* DatetimeIndex.to_datetime

All were deprecated in 0.19.0

xref pandas-devgh-8254, pandas-devgh-14096, pandas-devgh-14113
gfyoung added a commit that referenced this pull request Dec 6, 2017
Removes the following .to_datetime methods

* Index.to_datetime
* NaTType.to_datetime
* Timestamp.to_datetime
* PeriodIndex.to_datetime
* DatetimeIndex.to_datetime

All were deprecated in 0.19.0

xref gh-8254, gh-14096, gh-14113
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Deprecate Functionality to remove in pandas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants