-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Using corr with callable gives 1 on diagonals where the result should be NaN #25781
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
Labels
Algos
Non-arithmetic algos: value_counts, factorize, sorting, isin, clip, shift, diff
Enhancement
Numeric Operations
Arithmetic, Comparison, and Logical operations
Milestone
Comments
3 tasks
take |
peterpanmj
pushed a commit
to peterpanmj/pandas
that referenced
this issue
Sep 13, 2021
peterpanmj
pushed a commit
to peterpanmj/pandas
that referenced
this issue
Sep 14, 2021
peterpanmj
pushed a commit
to peterpanmj/pandas
that referenced
this issue
Sep 14, 2021
peterpanmj
pushed a commit
to peterpanmj/pandas
that referenced
this issue
Sep 14, 2021
4 tasks
peterpanmj
pushed a commit
to peterpanmj/pandas
that referenced
this issue
Sep 15, 2021
peterpanmj
pushed a commit
to peterpanmj/pandas
that referenced
this issue
Sep 16, 2021
peterpanmj
pushed a commit
to peterpanmj/pandas
that referenced
this issue
Sep 19, 2021
peterpanmj
pushed a commit
to peterpanmj/pandas
that referenced
this issue
Oct 9, 2021
peterpanmj
pushed a commit
to peterpanmj/pandas
that referenced
this issue
Oct 9, 2021
peterpanmj
pushed a commit
to peterpanmj/pandas
that referenced
this issue
Oct 11, 2021
closing from discussion in #43569 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Algos
Non-arithmetic algos: value_counts, factorize, sorting, isin, clip, shift, diff
Enhancement
Numeric Operations
Arithmetic, Comparison, and Logical operations
Code Sample, a copy-pastable example if possible
gives:
Problem description
This is a follow up on #25726. In the example above the self-correlation is returned as 1.0. However, it should actually be
nan
. I would argue the diagonal elements of the correlation matrix should be calculated instead of setting them to 1 here:pandas/pandas/core/frame.py
Lines 7025 to 7026 in cb00deb
df.corr()
gives the output I expected:Expected Output
Output of
pd.show_versions()
pandas: 0.24.2
pytest: 4.1.1
pip: 18.1
setuptools: 40.6.3
Cython: 0.29.3
numpy: 1.15.4
scipy: 1.2.0
pyarrow: None
xarray: None
IPython: 7.2.0
sphinx: 1.8.3
patsy: 0.5.1
dateutil: 2.7.5
pytz: 2018.9
blosc: None
bottleneck: None
tables: 3.4.4
numexpr: 2.6.9
feather: None
matplotlib: 3.0.2
openpyxl: 2.4.0-b1
xlrd: 1.2.0
xlwt: None
xlsxwriter: None
lxml.etree: 4.3.0
bs4: 4.7.1
html5lib: 1.0.1
sqlalchemy: None
pymysql: None
psycopg2: None
jinja2: 2.10
s3fs: None
fastparquet: None
pandas_gbq: None
pandas_datareader: None
gcsfs: None
The text was updated successfully, but these errors were encountered: