Skip to content

DEPR: some removals #12165

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 2 commits into from
Closed

DEPR: some removals #12165

wants to merge 2 commits into from

Conversation

jreback
Copy link
Contributor

@jreback jreback commented Jan 28, 2016

DEPR: Removal of cols keyword in favor of subset in DataFrame.duplicated() and DataFrame.drop_duplicates(), xref #6680

@jreback jreback added the Deprecate Functionality to remove in pandas label Jan 28, 2016
@jreback jreback added this to the 0.18.0 milestone Jan 28, 2016
@jreback
Copy link
Contributor Author

jreback commented Jan 28, 2016

removing the float indexers change as it is a rabbit hole :) (meaning a different PR!)

@@ -233,8 +233,6 @@ class Timestamp(_Timestamp):
Offset which Timestamp will have
tz : string, pytz.timezone, dateutil.tz.tzfile or None
Time zone for time which Timestamp will have.
unit : string
numpy unit used for conversion, if ts_input is int or float
Copy link
Member

Choose a reason for hiding this comment

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

Is there a reason you removed this?
As I think it is a valid kwarg?

In [155]: pd.Timestamp(5, unit='D')
Out[155]: Timestamp('1970-01-06 00:00:00')

Copy link
Contributor Author

Choose a reason for hiding this comment

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

shoot, you are right. was looking at something else. will revert

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

ok!

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.

2 participants