-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
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
DEPR: some removals #12165
Conversation
…ted() and DataFrame.drop_duplicates(), xref pandas-dev#6680
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 |
There was a problem hiding this comment.
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')
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok!
DEPR: Removal of cols keyword in favor of subset in DataFrame.duplicated() and DataFrame.drop_duplicates(), xref #6680