Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ubuntu 13.10 with mpl : test_hist_df* TypeError: barh() got multiple values for keyword argument 'bottom' #8648

Closed
yarikoptic opened this issue Oct 27, 2014 · 7 comments · Fixed by #8810
Closed

ubuntu 13.10 with mpl : test_hist_df* TypeError: barh() got multiple values for keyword argument 'bottom' #8648

yarikoptic opened this issue Oct 27, 2014 · 7 comments · Fixed by #8810
Labels
Testing pandas testing functions or related to the test suite Visualization plotting
Milestone
0.15.2

Comments

@yarikoptic
Copy link
Contributor

0.15.0

======================================================================
ERROR: test_hist_df (pandas.tests.test_graphics.TestDataFramePlots)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/buildd/pandas-0.15.0/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tests/test_graphics.py", line 2061, in test_hist_df
    axes = df.plot(kind='hist', rot=50, fontsize=8, orientation='horizontal')
  File "/tmp/buildd/pandas-0.15.0/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 2453, in plot_frame
    **kwds)
  File "/tmp/buildd/pandas-0.15.0/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 2293, in _plot
    plot_obj.generate()
  File "/tmp/buildd/pandas-0.15.0/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 920, in generate
    self._make_plot()
  File "/tmp/buildd/pandas-0.15.0/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 1949, in _make_plot
    artists = plotf(ax, y, column_num=i, **kwds)
  File "/tmp/buildd/pandas-0.15.0/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 1929, in plotf
    bottom=bottom, **kwds)
  File "/usr/lib/pymodules/python2.7/matplotlib/axes.py", line 8180, in hist
    color=c, bottom=bottom)
TypeError: barh() got multiple values for keyword argument 'bottom'

NSTALLED VERSIONS
------------------
commit: None
python: 2.7.5.final.0
python-bits: 64
OS: Linux
OS-release: 3.2.0-4-amd64
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: C
LANG: C

pandas: 0.15.0
nose: 1.3.0
Cython: 0.19
numpy: 1.7.1
scipy: 0.12.0
statsmodels: 0.5.0
IPython: None
sphinx: 1.1.3
patsy: 0.3.0
dateutil: 1.5
pytz: 2012c
bottleneck: None
tables: 2.4.0
numexpr: 2.0.1
matplotlib: 1.2.1
openpyxl: 1.7.0
xlrd: 0.9.2
xlwt: 0.7.4
xlsxwriter: None
lxml: None
bs4: 4.2.0
html5lib: 0.95-dev
httplib2: None
apiclient: None
rpy2: None
sqlalchemy: None
pymysql: None
psycopg2: None
@onesandzeroes
Copy link
Contributor

I'm not sure what the official word is on what versions of matplotlib pandas tries to support, but v1.2.1 is getting pretty old by now. I would try installing a more recent version through pip, e.g.:

sudo apt-get build-dep python-matplotlib
sudo pip install matplotlib

@jreback jreback added Testing pandas testing functions or related to the test suite Visualization plotting labels Oct 27, 2014
@jreback jreback added this to the 0.15.1 milestone Oct 27, 2014
@jreback
Copy link
Contributor

jreback commented Oct 27, 2014

@onesandzeroes this just needs a skip test for this function (for matplotlib < 1.3)
their are some examples already of how to do this IIRC.

want to do a PR?

we test with 1.3.1 on the 2.7_LOCALE build, so this is not picked up.

@onesandzeroes onesandzeroes mentioned this issue Nov 14, 2014
Merged
@yarikoptic
Copy link
Contributor Author

just tested build of v0.15.1-108-ge9ee47c ad it seems to be still there

======================================================================
ERROR: test_hist_df_coord (pandas.tests.test_graphics.TestDataFramePlots)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tests/test_graphics.py", line 2145, in test_hist
_df_coord
    ax = df.plot(kind='hist', bins=5, orientation='horizontal')
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 2454, in plot_frame
    **kwds)
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 2294, in _plot
    plot_obj.generate()
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 920, in generate
    self._make_plot()
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 1949, in _make_plot
    artists = plotf(ax, y, column_num=i, **kwds)
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 1929, in plotf
    bottom=bottom, **kwds)
  File "/usr/lib/pymodules/python2.7/matplotlib/axes.py", line 8180, in hist
    color=c, bottom=bottom)
TypeError: barh() got multiple values for keyword argument 'bottom'

======================================================================
ERROR: test_hist_kwargs (pandas.tests.test_graphics.TestSeriesPlots)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tests/test_graphics.py", line 821, in test_hist_
kwargs
    ax = self.ts.plot(kind='hist', orientation='horizontal')
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 2488, in plot_series
    **kwds)
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 2294, in _plot
    plot_obj.generate()
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 920, in generate
    self._make_plot()
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 1949, in _make_plot
    artists = plotf(ax, y, column_num=i, **kwds)
  File "/tmp/buildd/pandas-0.15.0+git108-ge9ee47c/debian/tmp/usr/lib/python2.7/dist-packages/pandas/tools/plotting.py", line 1929, in plotf
    bottom=bottom, **kwds)
  File "/usr/lib/pymodules/python2.7/matplotlib/axes.py", line 8180, in hist
    color=c, bottom=bottom)
TypeError: barh() got multiple values for keyword argument 'bottom'


INSTALLED VERSIONS
------------------
commit: None
python: 2.7.5.final.0
python-bits: 64
OS: Linux
OS-release: 3.2.0-4-amd64
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: C
LANG: C

pandas: 0.15.1.dev
nose: 1.3.0
Cython: 0.19
numpy: 1.7.1
scipy: 0.12.0
statsmodels: 0.5.0
IPython: None
sphinx: 1.1.3
patsy: 0.3.0
dateutil: 1.5
pytz: 2012c
bottleneck: None
tables: 2.4.0
numexpr: 2.0.1
matplotlib: 1.2.1
openpyxl: 1.7.0
xlrd: 0.9.2
xlwt: 0.7.4
xlsxwriter: None
lxml: None
bs4: 4.2.0
html5lib: 0.95-dev
httplib2: None
apiclient: None
rpy2: None
sqlalchemy: None
pymysql: None
psycopg2: None

that was in ubuntu 13.10

@jreback
Copy link
Contributor

jreback commented Nov 30, 2014

could be, we don't test on that old matplotlib. (only 1.3)

but running a branch to do exactly that. I'll create another issue if it shows up.

thxs

@TomAugspurger
@jorisvandenbossche

do we have a minimum supported matplotlib? (e.g. 1.2.1)? or is it >= 1.3?
cc @tcaswell

@jreback
Copy link
Contributor

jreback commented Nov 30, 2014

see #8947 (closing shortly with a PR anyhow)

@jorisvandenbossche
Copy link
Member

1.3 is only one year and some months old, so I would tend to say we should still support lower version (but then also test with that of course)

@jreback
Copy link
Contributor

jreback commented Nov 30, 2014

ok, maybe we should mention that 1.2.1 is minimum (we are now passing with that version), wasn't tested before

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Testing pandas testing functions or related to the test suite Visualization plotting
Projects
None yet
Milestone
0.15.2
Development

Successfully merging a pull request may close this issue.

TST: Skip hist testing for matplotlib <= 1.2 onesandzeroes/pandas
4 participants
@yarikoptic @jreback @jorisvandenbossche @onesandzeroes