Skip to content

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 <[email protected]>
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 <[email protected]>
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

Development

Successfully merging this pull request may close these issues.

BUG: The behavior of DataFrame.corrwith against Series is changed? BUG: The behavior for DataFrame.corrwith is changed in pandas 1.5.0 ??

3 participants