Skip to content

Commit

Permalink
address reviews
Browse files Browse the repository at this point in the history
  • Loading branch information
galipremsagar committed Sep 7, 2021
1 parent c0435e1 commit cfac5b9
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
3 changes: 1 addition & 2 deletions python/cudf/cudf/tests/test_dataframe.py
Original file line number Diff line number Diff line change
Expand Up @@ -1241,8 +1241,7 @@ def test_dataframe_concat_different_numerical_columns(dtype1, dtype2):
else:
pres = pd.concat([df1, df2])
gres = cudf.concat([cudf.from_pandas(df1), cudf.from_pandas(df2)])
# Pandas 1.3.2+ returns mixed `object` dtype result.
assert_eq(cudf.from_pandas(pres.astype(gres.dtypes)), gres)
assert_eq(pres, gres, check_dtype=False)


def test_dataframe_concat_different_column_types():
Expand Down
9 changes: 3 additions & 6 deletions python/cudf/cudf/tests/test_index.py
Original file line number Diff line number Diff line change
Expand Up @@ -1873,12 +1873,9 @@ def test_index_fillna(data, fill_value):
pdi = pd.Index(data)
gdi = cudf.Index(data)

if isinstance(gdi, cudf.Int64Index) and isinstance(pdi, pd.Float64Index):
assert_eq(
pdi.fillna(fill_value).astype(gdi.dtype), gdi.fillna(fill_value)
)
else:
assert_eq(pdi.fillna(fill_value), gdi.fillna(fill_value))
assert_eq(
pdi.fillna(fill_value), gdi.fillna(fill_value), exact=False
) # Int64Index v/s Float64Index


@pytest.mark.parametrize(
Expand Down

0 comments on commit cfac5b9

Please sign in to comment.