-
Notifications
You must be signed in to change notification settings - Fork 890
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade pandas pinnings & make style fixes #13599
Upgrade pandas pinnings & make style fixes #13599
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me. As usual not expecting tests to pass on this feature branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving dask-cudf changes
@@ -2891,7 +2891,7 @@ def test_rangeindex_join_user_option(default_integer_bitwidth): | |||
actual = idx1.join(idx2, how="inner", sort=True) | |||
expected = idx1.to_pandas().join(idx2.to_pandas(), how="inner", sort=True) | |||
assert actual.dtype == cudf.dtype(f"int{default_integer_bitwidth}") | |||
assert_eq(expected, actual) | |||
assert_eq(expected, actual, exact=False) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the exactness not possible because the bitwidth might be different?
@@ -183,7 +183,9 @@ def test_dataframe_join_suffix(): | |||
assert_eq(expect.index.values, got.index.values) | |||
|
|||
got_sorted = got.sort_values(by=list(got.columns), axis=0) | |||
expect.index.name = None |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this an API-mismatch in sort_values
for us?
@galipremsagar do we need to move this PR forward? It looks like #13624 was created just to test this against the latest main branch? |
#14918 solves this issues being addressed in this PR |
Description
This PR upgrades pandas pinning & fixes style issues in the feature branch.
Checklist