Skip to content

Commit

Permalink
[REVIEW] Fixing dask tsvd stress test failure (#2941)
Browse files Browse the repository at this point in the history
* ADD int conversion

* UPDATE changelog.md

* ADD int to params

Co-authored-by: Nanthini Balasubramanian <[email protected]>
  • Loading branch information
Nanthini10 and Nanthini10 authored Oct 15, 2020
1 parent 0dc18e0 commit 5aa17ac
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,7 @@
- PR #2932: Marking KBinsDiscretizer pytests as xfail
- PR #2925: Fixing Owner Bug When Slicing CumlArray Objects
- PR #2931: Fix notebook error handling in gpuCI
- PR #2941: Fixing dask tsvd stress test failure
- PR #2943: Remove unused shuffle_features parameter
- PR #2940: Correcting labels meta dtype for `cuml.dask.make_classification`
- PR #2965: Notebooks update
Expand Down
6 changes: 3 additions & 3 deletions python/cuml/test/dask/test_tsvd.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

@pytest.mark.mg
@pytest.mark.parametrize("data_info", [unit_param([1000, 20, 30]),
stress_param([9e6, 5000, 30])])
stress_param([int(9e6), 5000, 30])])
@pytest.mark.parametrize("input_type", ["dataframe", "array"])
def test_pca_fit(data_info, input_type, client):

Expand Down Expand Up @@ -72,7 +72,7 @@ def test_pca_fit(data_info, input_type, client):

@pytest.mark.mg
@pytest.mark.parametrize("data_info", [unit_param([1000, 20, 46]),
stress_param([9e6, 5000, 46])])
stress_param([int(9e6), 5000, 46])])
def test_pca_fit_transform_fp32(data_info, client):

nrows, ncols, n_parts = data_info
Expand All @@ -92,7 +92,7 @@ def test_pca_fit_transform_fp32(data_info, client):

@pytest.mark.mg
@pytest.mark.parametrize("data_info", [unit_param([1000, 20, 33]),
stress_param([9e6, 5000, 33])])
stress_param([int(9e6), 5000, 33])])
def test_pca_fit_transform_fp64(data_info, client):

nrows, ncols, n_parts = data_info
Expand Down

0 comments on commit 5aa17ac

Please sign in to comment.