CLN: Remove pandas.tools module (pandas-dev#23376)

xref pandas-devgh-15358.
xref pandas-devgh-16005.

Author: gfyoung

doc/source/whatsnew/v0.24.0.txt

@@ -953,6 +953,7 @@ Removal of prior version deprecations/changes
 - :meth:`TimedeltaIndex.searchsorted`, :meth:`DatetimeIndex.searchsorted`, and :meth:`PeriodIndex.searchsorted` have renamed the ``key`` argument to ``value`` (:issue:`14645`)
 - :meth:`DataFrame.consolidate` and :meth:`Series.consolidate` have been removed (:issue:`15501`)
 - Removal of the previously deprecated module ``pandas.json`` (:issue:`19944`)
+- The module ``pandas.tools`` has been removed (:issue:`15358`, :issue:`16005`)
 - :meth:`SparseArray.get_values` and :meth:`SparseArray.to_dense` have dropped the ``fill`` parameter (:issue:`14686`)
 - :meth:`DataFrame.sortlevel` and :meth:`Series.sortlevel` have been removed (:issue:`15099`)
 - :meth:`SparseSeries.to_dense` has dropped the ``sparse_only`` parameter (:issue:`14686`)

pandas/__init__.py

@@ -45,14 +45,6 @@
 from pandas.core.computation.api import *
 from pandas.core.reshape.api import *
 
-# deprecate tools.plotting, plot_params and scatter_matrix on the top namespace
-import pandas.tools.plotting
-plot_params = pandas.plotting._style._Options(deprecated=True)
-# do not import deprecate to top namespace
-scatter_matrix = pandas.util._decorators.deprecate(
-    'pandas.scatter_matrix', pandas.plotting.scatter_matrix, '0.20.0',
-    'pandas.plotting.scatter_matrix')
-
 from pandas.util._print_versions import show_versions
 from pandas.io.api import *
 from pandas.util._tester import test

pandas/plotting/_style.py

@@ -110,22 +110,14 @@ def __init__(self, deprecated=False):
         # self['xaxis.compat'] = False
         super(_Options, self).__setitem__('xaxis.compat', False)
 
-    def _warn_if_deprecated(self):
-        if self._deprecated:
-            warnings.warn("'pandas.plot_params' is deprecated. Use "
-                          "'pandas.plotting.plot_params' instead",
-                          FutureWarning, stacklevel=3)
-
     def __getitem__(self, key):
-        self._warn_if_deprecated()
         key = self._get_canonical_key(key)
         if key not in self:
             raise ValueError(
                 '{key} is not a valid pandas plotting option'.format(key=key))
         return super(_Options, self).__getitem__(key)
 
     def __setitem__(self, key, value):
-        self._warn_if_deprecated()
         key = self._get_canonical_key(key)
         return super(_Options, self).__setitem__(key, value)
 
@@ -148,7 +140,6 @@ def reset(self):
         -------
         None
         """
-        self._warn_if_deprecated()
         self.__init__()
 
     def _get_canonical_key(self, key):
@@ -160,7 +151,6 @@ def use(self, key, value):
         Temporarily set a parameter value using the with statement.
         Aliasing allowed.
         """
-        self._warn_if_deprecated()
         old_value = self[key]
         try:
             self[key] = value

pandas/tests/api/test_api.py

@@ -29,7 +29,7 @@ class TestPDApi(Base):
 
     # top-level sub-packages
     lib = ['api', 'compat', 'core', 'errors', 'pandas',
-           'plotting', 'test', 'testing', 'tools', 'tseries',
+           'plotting', 'test', 'testing', 'tseries',
            'util', 'options', 'io']
 
     # these are already deprecated; awaiting removal
@@ -88,7 +88,7 @@ class TestPDApi(Base):
     deprecated_funcs_in_future = []
 
     # these are already deprecated; awaiting removal
-    deprecated_funcs = ['plot_params', 'scatter_matrix']
+    deprecated_funcs = []
 
     def test_api(self):