-
-
Notifications
You must be signed in to change notification settings - Fork 18k
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: Series.put, Series.real, Series.imag, Index.dtype_str #27106
Conversation
Hello @mroeschke! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2019-06-28 21:44:10 UTC |
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.
lgtm, we should use FutureWarning here
pandas/core/series.py
Outdated
""" | ||
warnings.warn("`imag` has be deprecated and will be removed in a " | ||
"future verison", DeprecationWarning, stacklevel=2) |
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.
DeprecationWarning -> FutureWaring on all of these
doc/source/whatsnew/v0.25.0.rst
Outdated
@@ -604,6 +604,9 @@ Other deprecations | |||
- The :meth:`Series.ftype`, :meth:`Series.ftypes` and :meth:`DataFrame.ftypes` methods are deprecated and will be removed in a future version. | |||
Instead, use :meth:`Series.dtype` and :meth:`DataFrame.dtypes` (:issue:`26705`). | |||
- :meth:`Timedelta.resolution` is deprecated and replaced with :meth:`Timedelta.resolution_string`. In a future version, :meth:`Timedelta.resolution` will be changed to behave like the standard library :attr:`timedelta.resolution` (:issue:`21344`) | |||
- :attr:`Index.dtype_str` is deprecated. (:issue:`18262`) | |||
- :attr:`Series.imag` and :attr:`Series.real` is deprecated. (:issue:`18262`) |
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 -> are
can you add to the list in #6581 as well |
Changed to FutureWarnings and green. |
thanks @mroeschke |
git diff upstream/master -u -- "*.py" | flake8 --diff
Removes:
Series.put
Index.dtype_str
Series.real
Series.imag