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.

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

Revert "PERF: faster corrwith method for pearson and spearman correlation when other is a Series and axis = 0 (column-wise) (#46174)" #49140

Merged
merged 6 commits into from Oct 17, 2022

Conversation

MarcoGorelli
Copy link
Member

@MarcoGorelli MarcoGorelli commented Oct 17, 2022

Reverting for now - I've a test fix and whatsnew note from #49032 , but have reverted the change and added the issue as a test so we can go ahead with 1.5.1. A fixed version of the performance improvement can go into 2.0.0

@mroeschke mroeschke added the Algos Non-arithmetic algos: value_counts, factorize, sorting, isin, clip, shift, diff label Oct 17, 2022
@mroeschke mroeschke added this to the 1.5.1 milestone Oct 17, 2022
@mroeschke
Copy link
Member

Thanks @MarcoGorelli

meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Oct 17, 2022
…r pearson and spearman correlation when other is a Series and axis = 0 (column-wise) (pandas-dev#46174)"
MarcoGorelli added a commit that referenced this pull request Oct 17, 2022
…hod for pearson and spearman correlation when other is a Series and axis = 0 (column-wise) (#46174)") (#49151)

Backport PR #49140: Revert "PERF: faster corrwith method for pearson and spearman correlation when other is a Series and axis = 0 (column-wise) (#46174)"

Co-authored-by: Marco Edward Gorelli <33491632+MarcoGorelli@users.noreply.github.com>
noatamir pushed a commit to noatamir/pandas that referenced this pull request Nov 9, 2022
…tion when other is a Series and axis = 0 (column-wise) (pandas-dev#46174)" (pandas-dev#49140)

* Update frame.py

* Revert "PERF: faster corrwith method for pearson and spearman correlation when other is a Series and axis = 0 (column-wise) (pandas-dev#46174)"

This reverts commit 5efb570.

* fix GH issue number in test

* add test from original issue

* skip if no scipy

* add extra test case

Co-authored-by: Yuanhao Geng <41546976+GYHHAHA@users.noreply.github.com>
Co-authored-by: MarcoGorelli <>
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
Projects
None yet
3 participants