Skip to content

Commit

Permalink
Merge pull request #1 from dominicshanshan/dominicshanshan-cudf-dataf…
Browse files Browse the repository at this point in the history
…rame-add-spearman

[REVIEW]Add 'spearman' correlation matrix in dataframe.py rapidsai#6804
  • Loading branch information
dominicshanshan authored Jan 14, 2021
2 parents e0e2cf8 + 78e17f8 commit e62b07f
Showing 1 changed file with 16 additions and 6 deletions.
22 changes: 16 additions & 6 deletions python/cudf/cudf/core/dataframe.py
Original file line number Diff line number Diff line change
Expand Up @@ -6931,12 +6931,22 @@ def cov(self, **kwargs):
df.columns = self.columns
return df

def corr(self):
"""Compute the correlation matrix of a DataFrame.
"""
corr = cupy.corrcoef(self.values, rowvar=False)
df = DataFrame(cupy.asfortranarray(corr)).set_index(self.columns)
df.columns = self.columns
def corr(self, method):
"""Compute the correlation matrix of a DataFrame. Method: {'pearson', 'spearman'}
"""
if method == 'pearson':
corr = cupy.corrcoef(self.values, rowvar=False)
df = DataFrame(cupy.asfortranarray(corr)).set_index(self.columns)
df.columns = self.columns

elif method == 'spearman':
corr = cupy.corrcoef(self.rank().values, rowvar=False)
df = DataFrame(cupy.asfortranarray(corr)).set_index(self.columns)
df.columns = self.columns

else:
raise ValueError("method must be either 'pearson', 'spearman'")

return df

def to_dict(self, orient="dict", into=dict):
Expand Down

0 comments on commit e62b07f

Please sign in to comment.