Move tests upder plotting and other corrections

Author: sinhrks

pandas/__init__.py

@@ -50,9 +50,13 @@
                                 merge_ordered, merge_asof)
 from pandas.tools.pivot import pivot_table, crosstab
 
-# deprecated
+# deprecate tools.plotting, and directly imported scatter_matrix
 import pandas.tools.plotting
-from pandas.plotting import scatter_matrix, plot_params
+from pandas.plotting import plot_params
+from pandas.util.decorators import deprecate
+scatter_matrix = deprecate('pandas.scatter_matrix', pandas.plotting.scatter_matrix,
+                           'pandas.plotting.scatter_matrix')
+
 from pandas.tools.tile import cut, qcut
 from pandas.tools.util import to_numeric
 from pandas.core.reshape import melt

pandas/core/frame.py

@@ -90,7 +90,7 @@
 import pandas.core.ops as ops
 import pandas.formats.format as fmt
 from pandas.formats.printing import pprint_thing
-import pandas.plotting.plotting as gfx
+import pandas.plotting.core as gfx
 
 from pandas._libs import lib, algos as libalgos
 

pandas/core/groupby.py

@@ -4159,7 +4159,7 @@ def groupby_series(obj, col=None):
         return results
 
 
-from pandas.plotting.plotting import boxplot_frame_groupby  # noqa
+from pandas.plotting.core import boxplot_frame_groupby  # noqa
 DataFrameGroupBy.boxplot = boxplot_frame_groupby
 
 

pandas/core/series.py

@@ -3001,7 +3001,7 @@ def create_from_value(value, index, dtype):
 # ----------------------------------------------------------------------
 # Add plotting methods to Series
 
-import pandas.plotting.plotting as _gfx  # noqa
+import pandas.plotting.core as _gfx  # noqa
 
 Series.plot = base.AccessorProperty(_gfx.SeriesPlotMethods,
                                     _gfx.SeriesPlotMethods)

pandas/plotting/api.py

@@ -5,16 +5,16 @@
 # flake8: noqa
 
 try:  # mpl optional
-    from pandas.plotting import converter as conv
-    conv.register()  # needs to override so set_xlim works with str/number
+    from pandas.plotting import converter
+    converter.register()  # needs to override so set_xlim works with str/number
 except ImportError:
     pass
 
 from pandas.plotting.misc import (scatter_matrix, radviz,
                                   andrews_curves, bootstrap_plot,
                                   parallel_coordinates, lag_plot,
                                   autocorrelation_plot)
-from pandas.plotting.plotting import (boxplot, scatter_plot, grouped_hist,
-                                      hist_frame, hist_series)
+from pandas.plotting.core import (boxplot, scatter_plot, grouped_hist,
+                                  hist_frame, hist_series)
 from pandas.plotting.style import plot_params
-from pandas.plotting.tools import table
+from pandas.plotting.tools import table

pandas/plotting/plotting.py renamed to pandas/plotting/core.py

@@ -73,7 +73,8 @@ def setUp(self):
         self.default_tick_position = 'left' if self.mpl_ge_2_0_0 else 'default'
         # common test data
         from pandas import read_csv
-        path = os.path.join(os.path.dirname(curpath()), 'data', 'iris.csv')
+        base = os.path.join(os.path.dirname(curpath()), os.pardir)
+        path = os.path.join(base, 'tests', 'data', 'iris.csv')
         self.iris = read_csv(path)
 
         n = 100

pandas/tests/plotting/__init__.py renamed to pandas/plotting/tests/__init__.py

@@ -14,9 +14,9 @@
 from numpy import random
 from numpy.random import randn
 
-import pandas.tools.plotting as plotting
+import pandas.plotting as plotting
 
-from pandas.tests.plotting.common import (TestPlotBase, _check_plot_works)
+from pandas.plotting.tests.common import (TestPlotBase, _check_plot_works)
 
 
 """ Test cases for .boxplot method """

pandas/tests/plotting/common.py renamed to pandas/plotting/tests/common.py

@@ -17,7 +17,7 @@
 from pandas.util.testing import assert_series_equal, ensure_clean, slow
 import pandas.util.testing as tm
 
-from pandas.tests.plotting.common import (TestPlotBase,
+from pandas.plotting.tests.common import (TestPlotBase,
                                           _skip_if_no_scipy_gaussian_kde)
 
 

pandas/tests/plotting/test_boxplot_method.py renamed to pandas/plotting/tests/test_boxplot_method.py

@@ -0,0 +1,68 @@
+# coding: utf-8
+
+import nose
+import string
+
+import pandas as pd
+import pandas.util.testing as tm
+from pandas.util.testing import slow
+
+import numpy as np
+from numpy.random import randn
+
+import pandas.tools.plotting as plotting
+
+from pandas.plotting.tests.common import TestPlotBase
+
+
+"""
+Test cases for plot functions imported from deprecated
+pandas.tools.plotting
+"""
+
+
+@tm.mplskip
+class TestDeprecatedNameSpace(TestPlotBase):
+
+    @slow
+    def test_scatter_plot_legacy(self):
+        tm._skip_if_no_scipy()
+
+        df = pd.DataFrame(randn(100, 2))
+
+        with tm.assert_produces_warning(FutureWarning):
+            plotting.scatter_matrix(df)
+
+        with tm.assert_produces_warning(FutureWarning):
+            pd.scatter_matrix(df)
+
+    @slow
+    def test_boxplot_deprecated(self):
+        df = pd.DataFrame(randn(6, 4),
+                          index=list(string.ascii_letters[:6]),
+                          columns=['one', 'two', 'three', 'four'])
+        df['indic'] = ['foo', 'bar'] * 3
+
+        with tm.assert_produces_warning(FutureWarning):
+            plotting.boxplot(df, column=['one', 'two'],
+                             by='indic')
+
+    @slow
+    def test_grouped_hist_legacy(self):
+        df = pd.DataFrame(randn(500, 2), columns=['A', 'B'])
+        df['C'] = np.random.randint(0, 4, 500)
+        df['D'] = ['X'] * 500
+
+        with tm.assert_produces_warning(FutureWarning):
+            plotting.grouped_hist(df.A, by=df.C)
+
+    @slow
+    def test_radviz_deprecated(self):
+        df = self.iris
+        with tm.assert_produces_warning(FutureWarning):
+            plotting.radviz(frame=df, class_column='Name')
+
+
+if __name__ == '__main__':
+    nose.runmodule(argv=[__file__, '-vvs', '-x', '--pdb', '--pdb-failure'],
+                   exit=False)

pandas/tests/plotting/test_converter.py renamed to pandas/plotting/tests/test_converter.py

@@ -23,7 +23,7 @@
 from numpy.random import rand, randn
 
 import pandas.plotting as plotting
-from pandas.tests.plotting.common import (TestPlotBase, _check_plot_works,
+from pandas.plotting.tests.common import (TestPlotBase, _check_plot_works,
                                           _skip_if_no_scipy_gaussian_kde,
                                           _ok_for_gaussian_kde)
 
@@ -1979,7 +1979,7 @@ def test_unordered_ts(self):
 
     def test_kind_both_ways(self):
         df = DataFrame({'x': [1, 2, 3]})
-        for kind in plotting.plotting._common_kinds:
+        for kind in plotting.core._common_kinds:
             if not _ok_for_gaussian_kde(kind):
                 continue
             df.plot(kind=kind)
@@ -1990,7 +1990,7 @@ def test_kind_both_ways(self):
 
     def test_all_invalid_plot_data(self):
         df = DataFrame(list('abcd'))
-        for kind in plotting.plotting._common_kinds:
+        for kind in plotting.core._common_kinds:
             if not _ok_for_gaussian_kde(kind):
                 continue
             with tm.assertRaises(TypeError):
@@ -2001,7 +2001,7 @@ def test_partially_invalid_plot_data(self):
         with tm.RNGContext(42):
             df = DataFrame(randn(10, 2), dtype=object)
             df[np.random.rand(df.shape[0]) > 0.5] = 'a'
-            for kind in plotting.plotting._common_kinds:
+            for kind in plotting.core._common_kinds:
                 if not _ok_for_gaussian_kde(kind):
                     continue
                 with tm.assertRaises(TypeError):
@@ -2454,7 +2454,7 @@ def test_memory_leak(self):
         import gc
 
         results = {}
-        for kind in plotting.plotting._plot_klass.keys():
+        for kind in plotting.core._plot_klass.keys():
             if not _ok_for_gaussian_kde(kind):
                 continue
             args = {}
@@ -2653,7 +2653,7 @@ def test_df_grid_settings(self):
         # Make sure plot defaults to rcParams['axes.grid'] setting, GH 9792
         self._check_grid_settings(
             DataFrame({'a': [1, 2, 3], 'b': [2, 3, 4]}),
-            plotting.plotting._dataframe_kinds, kws={'x': 'a', 'y': 'b'})
+            plotting.core._dataframe_kinds, kws={'x': 'a', 'y': 'b'})
 
     def test_option_mpl_style(self):
         with tm.assert_produces_warning(FutureWarning,

pandas/tests/plotting/test_datetimelike.py renamed to pandas/plotting/tests/test_datetimelike.py

@@ -8,7 +8,7 @@
 
 import numpy as np
 
-from pandas.tests.plotting.common import TestPlotBase
+from pandas.plotting.tests.common import TestPlotBase
 
 
 @tm.mplskip

pandas/plotting/tests/test_deprecated.py

@@ -9,8 +9,8 @@
 import numpy as np
 from numpy.random import randn
 
-import pandas.tools.plotting as plotting
-from pandas.tests.plotting.common import (TestPlotBase, _check_plot_works)
+import pandas.plotting as plotting
+from pandas.plotting.tests.common import (TestPlotBase, _check_plot_works)
 
 
 @tm.mplskip

pandas/tests/plotting/test_frame.py renamed to pandas/plotting/tests/test_frame.py

@@ -11,8 +11,8 @@
 from numpy import random
 from numpy.random import randn
 
-import pandas.tools.plotting as plotting
-from pandas.tests.plotting.common import (TestPlotBase, _check_plot_works,
+import pandas.plotting as plotting
+from pandas.plotting.tests.common import (TestPlotBase, _check_plot_works,
                                           _ok_for_gaussian_kde)
 
 
@@ -29,7 +29,7 @@ def setUp(self):
 
     @slow
     def test_autocorrelation_plot(self):
-        from pandas.tools.plotting import autocorrelation_plot
+        from pandas.plotting import autocorrelation_plot
         _check_plot_works(autocorrelation_plot, series=self.ts)
         _check_plot_works(autocorrelation_plot, series=self.ts.values)
 
@@ -38,13 +38,13 @@ def test_autocorrelation_plot(self):
 
     @slow
     def test_lag_plot(self):
-        from pandas.tools.plotting import lag_plot
+        from pandas.plotting import lag_plot
         _check_plot_works(lag_plot, series=self.ts)
         _check_plot_works(lag_plot, series=self.ts, lag=5)
 
     @slow
     def test_bootstrap_plot(self):
-        from pandas.tools.plotting import bootstrap_plot
+        from pandas.plotting import bootstrap_plot
         _check_plot_works(bootstrap_plot, series=self.ts, size=10)
 
 
@@ -130,7 +130,7 @@ def test_scatter_matrix_axis(self):
 
     @slow
     def test_andrews_curves(self):
-        from pandas.tools.plotting import andrews_curves
+        from pandas.plotting import andrews_curves
         from matplotlib import cm
 
         df = self.iris
@@ -195,7 +195,7 @@ def test_andrews_curves(self):
 
     @slow
     def test_parallel_coordinates(self):
-        from pandas.tools.plotting import parallel_coordinates
+        from pandas.plotting import parallel_coordinates
         from matplotlib import cm
 
         df = self.iris
@@ -263,7 +263,7 @@ def test_parallel_coordinates_with_sorted_labels(self):
 
     @slow
     def test_radviz(self):
-        from pandas.tools.plotting import radviz
+        from pandas.plotting import radviz
         from matplotlib import cm
 
         df = self.iris

pandas/tests/plotting/test_groupby.py renamed to pandas/plotting/tests/test_groupby.py

@@ -17,7 +17,7 @@
 from numpy.random import randn
 
 import pandas.plotting as plotting
-from pandas.tests.plotting.common import (TestPlotBase, _check_plot_works,
+from pandas.plotting.tests.common import (TestPlotBase, _check_plot_works,
                                           _skip_if_no_scipy_gaussian_kde,
                                           _ok_for_gaussian_kde)
 
@@ -622,8 +622,8 @@ def test_boxplot_series(self):
     @slow
     def test_kind_both_ways(self):
         s = Series(range(3))
-        kinds = (plotting.plotting._common_kinds +
-                 plotting.plotting._series_kinds)
+        kinds = (plotting.core._common_kinds +
+                 plotting.core._series_kinds)
         for kind in kinds:
             if not _ok_for_gaussian_kde(kind):
                 continue
@@ -633,7 +633,7 @@ def test_kind_both_ways(self):
     @slow
     def test_invalid_plot_data(self):
         s = Series(list('abcd'))
-        for kind in plotting.plotting._common_kinds:
+        for kind in plotting.core._common_kinds:
             if not _ok_for_gaussian_kde(kind):
                 continue
             with tm.assertRaises(TypeError):
@@ -642,14 +642,14 @@ def test_invalid_plot_data(self):
     @slow
     def test_valid_object_plot(self):
         s = Series(lrange(10), dtype=object)
-        for kind in plotting.plotting._common_kinds:
+        for kind in plotting.core._common_kinds:
             if not _ok_for_gaussian_kde(kind):
                 continue
             _check_plot_works(s.plot, kind=kind)
 
     def test_partially_invalid_plot_data(self):
         s = Series(['a', 'b', 1.0, 2])
-        for kind in plotting.plotting._common_kinds:
+        for kind in plotting.core._common_kinds:
             if not _ok_for_gaussian_kde(kind):
                 continue
             with tm.assertRaises(TypeError):
@@ -720,8 +720,8 @@ def test_table(self):
     def test_series_grid_settings(self):
         # Make sure plot defaults to rcParams['axes.grid'] setting, GH 9792
         self._check_grid_settings(Series([1, 2, 3]),
-                                  plotting.plotting._series_kinds +
-                                  plotting.plotting._common_kinds)
+                                  plotting.core._series_kinds +
+                                  plotting.core._common_kinds)
 
     @slow
     def test_standard_colors(self):

pandas/tests/plotting/test_hist_method.py renamed to pandas/plotting/tests/test_hist_method.py

@@ -134,7 +134,7 @@ def _replot_ax(ax, freq, kwargs):
 
             # for tsplot
             if isinstance(plotf, compat.string_types):
-                from pandas.tools.plotting import _plot_klass
+                from pandas.plotting.core import _plot_klass
                 plotf = _plot_klass[plotf]._plot
 
             lines.append(plotf(ax, series.index._mpl_repr(),

pandas/tests/plotting/test_misc.py renamed to pandas/plotting/tests/test_misc.py

@@ -0,0 +1,21 @@
+import sys
+import warnings
+
+import pandas.plotting.api as api
+
+# back-compat of public API
+# deprecate these functions
+m = sys.modules['pandas.tools.plotting']
+for t in [t for t in dir(api) if not t.startswith('_')]:
+
+    def outer(t=t):
+
+        def wrapper(*args, **kwargs):
+            warnings.warn("pandas.tools.plotting.{t} is deprecated. "
+                          "import from the "
+                          "pandas.plotting.{t} instead".format(t=t),
+                          FutureWarning, stacklevel=2)
+            return getattr(api, t)(*args, **kwargs)
+        return wrapper
+
+    setattr(m, t, outer(t))

pandas/tests/plotting/test_series.py renamed to pandas/plotting/tests/test_series.py

@@ -131,7 +131,7 @@ def _make_table(self, ax, df, title, height=None):
             ax.set_visible(False)
             return
 
-        import pandas.tools.plotting as plotting
+        import pandas.plotting as plotting
 
         idx_nlevels = df.index.nlevels
         col_nlevels = df.columns.nlevels