BUG: don't clobber color keyword in Series.plot, close #1890

Author: wesm

RELEASE.rst

@@ -148,6 +148,7 @@ pandas 0.9.0
   - Fix resampling error with intraday times and anchored target time (like
     AS-DEC) (#1772)
   - Fix .ix indexing bugs with mixed-integer indexes (#1799)
+  - Respect passed color keyword argument in Series.plot (#1890)
 
 pandas 0.8.1
 ============

pandas/tests/test_graphics.py

@@ -479,6 +479,16 @@ def test_boxplot(self):
         _check_plot_works(grouped.boxplot)
         _check_plot_works(grouped.boxplot, subplots=False)
 
+    @slow
+    def test_series_plot_color_kwargs(self):
+        # #1890
+        import matplotlib.pyplot as plt
+
+        plt.close('all')
+        ax = Series(np.arange(12) + 1).plot(color='green')
+        line = ax.get_lines()[0]
+        self.assert_(line.get_color() == 'green')
+
 PNG_PATH = 'tmp.png'
 
 def _check_plot_works(f, *args, **kwargs):

pandas/tools/plotting.py

@@ -889,7 +889,8 @@ def _make_plot(self):
             has_colors, colors = self._get_colors()
             def _maybe_add_color(kwargs, style, i):
                 if (not has_colors and
-                    (style is None or re.match('[a-z]+', style) is None)):
+                    (style is None or re.match('[a-z]+', style) is None)
+                    and 'color' not in kwargs):
                     kwargs['color'] = colors[i % len(colors)]
 
             plotf = self._get_plot_function()