Skip to content

BUG: Pandas-1.3.4 (and1.3.2, and 1.3.1) do not work well with seaborn-0.11.2 on PyPy-3.7-7.3.7 , but pandas-1.2.5 does #44253

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.

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

Closed
2 of 3 tasks
stonebig opened this issue Oct 31, 2021 · 1 comment · Fixed by #44879
Labels
Bug Compat pandas objects compatability with Numpy or Python functions IO CSV read_csv, to_csv PyPy
Milestone

Comments

@stonebig
Copy link
Contributor

stonebig commented Oct 31, 2021

  • I have checked that this issue has not already been reported.

  • I have confirmed this bug exists on the latest version of pandas.

  • I have confirmed this bug exists on the master branch of pandas.

Reproducible Example

with pypy3.7-v7.3.7-win64.zip and wheels provided per cgohlke:

  • the following example works with seaborn-0.11.2 (and 0.11.1) and Pandas-1.2.5,
  • but fails with Pandas-1.3.1, 1.3.2 and latest 1.3.4
import seaborn as sns
sns.set()
df = sns.load_dataset("iris")
sns.pairplot(df, hue="species", height=1.5)

Issue Description

jupyterlab kernel crashes and restarts,

https://github.com/mwaskom/seaborn/blob/master/seaborn/utils.py#L586

image

image

Expected Behavior

a typical seaborn graph, (when pandas-1.2.5)
image

also this pure pandas one works

import pandas as pd
df = pd.read_csv('~/seaborn-data/iris.csv',index_col=False)

import pandas as pd
import altair as alt

alt.Chart(df).mark_bar().encode(
    x=alt.X('sepal_length', bin=alt.Bin(maxbins=50)),
    y='count(*):Q',
    color='species:N',
    #column='species',
).interactive()

image

Installed Versions

INSTALLED VERSIONS

commit : 945c9ed
python : 3.7.12.final.0
python-bits : 64
OS : Windows
OS-release : 10
Version : 10.0.19043
machine : AMD64
processor : Intel64 Family 6 Model 58 Stepping 9, GenuineIntel
byteorder : little
LC_ALL : None
LANG : None
LOCALE : None.None

pandas : 1.3.4
numpy : 1.21.3
pytz : 2021.3
dateutil : 2.8.2
pip : 21.3.1
setuptools : 58.3.0
Cython : 0.29.24
pytest : 6.2.5
hypothesis : 6.24.0
sphinx : 4.2.0
blosc : 1.10.6
feather : None
xlsxwriter : 3.0.1
lxml.etree : 4.6.3
html5lib : 1.1
pymysql : None
psycopg2 : None
jinja2 : 3.0.2
IPython : 7.29.0
pandas_datareader: 0.10.0
bs4 : 4.10.0
bottleneck : 1.3.2
fsspec : 2021.10.1
fastparquet : None
gcsfs : None
matplotlib : 3.4.3
numexpr : None
odfpy : None
openpyxl : 3.0.9
pandas_gbq : None
pyarrow : None
pyxlsb : None
s3fs : None
scipy : 1.7.1
sqlalchemy : 1.4.26
tables : None
tabulate : 0.8.9
xarray : 0.19.0
xlrd : None
xlwt : None
numba : None

@stonebig stonebig added Bug Needs Triage Issue that has not been reviewed by a pandas team member labels Oct 31, 2021
@stonebig stonebig changed the title BUG: Pandas-1.3.4 (and1.3.2, and 1.3.1) do not work well with seaborn-0.11.2 on PyPy-3.7-7.3.7 BUG: Pandas-1.3.4 (and1.3.2, and 1.3.1) do not work well with seaborn-0.11.2 on PyPy-3.7-7.3.7 , but pandas-1.2.5 does Oct 31, 2021
@lithomas1
Copy link
Member

Thanks for reporting this. We have 0 testing on PyPy unfortunately(adding that testing is in a separate issue, and last time I checked we had a ton of segfaults/test failures). However, if you can find a patch for this, we would be happy to take it.

@lithomas1 lithomas1 added Compat pandas objects compatability with Numpy or Python functions IO CSV read_csv, to_csv PyPy and removed Needs Triage Issue that has not been reviewed by a pandas team member labels Nov 1, 2021
@lithomas1 lithomas1 added this to the Contributions Welcome milestone Nov 1, 2021
@jreback jreback modified the milestones: Contributions Welcome, 1.4 Dec 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Compat pandas objects compatability with Numpy or Python functions IO CSV read_csv, to_csv PyPy
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants