Merge pull request pandas-dev#10962 from sinhrks/plot_dupcolumns

BUG: DataFrame subplot with duplicated columns output incorrect result

Author: sinhrks

doc/source/whatsnew/v0.17.0.txt

@@ -927,3 +927,4 @@ Bug Fixes
 - Bug in ``groupby`` incorrect computation for aggregation on ``DataFrame`` with ``NaT`` (E.g ``first``, ``last``, ``min``). (:issue:`10590`)
 - Bug when constructing ``DataFrame`` where passing a dictionary with only scalar values and specifying columns did not raise an error (:issue:`10856`)
 - Bug in ``.var()`` causing roundoff errors for highly similar values (:issue:`10242`)
+- Bug in ``DataFrame.plot(subplots=True)`` with duplicated columns outputs incorrect result (:issue:`10962`)

pandas/tests/test_graphics.py

@@ -1646,6 +1646,28 @@ def test_subplots_sharex_axes_existing_axes(self):
         for ax in axes.ravel():
             self._check_visible(ax.get_yticklabels(), visible=True)
 
+    @slow
+    def test_subplots_dup_columns(self):
+        # GH 10962
+        df = DataFrame(np.random.rand(5, 5), columns=list('aaaaa'))
+        axes = df.plot(subplots=True)
+        for ax in axes:
+            self._check_legend_labels(ax, labels=['a'])
+            self.assertEqual(len(ax.lines), 1)
+        tm.close()
+
+        axes = df.plot(subplots=True, secondary_y='a')
+        for ax in axes:
+            # (right) is only attached when subplots=False
+            self._check_legend_labels(ax, labels=['a'])
+            self.assertEqual(len(ax.lines), 1)
+        tm.close()
+
+        ax = df.plot(secondary_y='a')
+        self._check_legend_labels(ax, labels=['a (right)'] * 5)
+        self.assertEqual(len(ax.lines), 0)
+        self.assertEqual(len(ax.right_ax.lines), 5)
+
     def test_negative_log(self):
         df = - DataFrame(rand(6, 4),
                        index=list(string.ascii_letters[:6]),

pandas/tools/plotting.py

@@ -920,11 +920,11 @@ def _iter_data(self, data=None, keep_index=False, fillna=None):
         else:
             columns = data.columns
 
-        for col in columns:
+        for col, values in data.iteritems():
             if keep_index is True:
-                yield col, data[col]
+                yield col, values
             else:
-                yield col, data[col].values
+                yield col, values.values
 
     @property
     def nseries(self):