Skip to content

Commit

Permalink
Add scipy skip for a test (#10502)
Browse files Browse the repository at this point in the history
This PR skips `scipy` related code-path that pandas takes. This isn't a failure in CI, but cudf dev env doesn't require having a `scipy` package, hence adding an `importorskip`.

Authors:
  - GALI PREM SAGAR (https://github.com/galipremsagar)

Approvers:
  - Vyas Ramasubramani (https://github.com/vyasr)
  - Ashwin Srinath (https://github.com/shwina)

URL: #10502
  • Loading branch information
galipremsagar authored Mar 24, 2022
1 parent 12b66a3 commit 54918d8
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions python/cudf/cudf/tests/test_stats.py
Original file line number Diff line number Diff line change
Expand Up @@ -423,6 +423,10 @@ def test_cov1d(data1, data2):
)
@pytest.mark.parametrize("method", ["spearman", "pearson"])
def test_corr1d(data1, data2, method):
if method == "spearman":
# Pandas uses scipy.stats.spearmanr code-path
pytest.importorskip("scipy")

gs1 = cudf.Series(data1)
gs2 = cudf.Series(data2)

Expand Down

0 comments on commit 54918d8

Please sign in to comment.