-
Notifications
You must be signed in to change notification settings - Fork 912
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
Remove mr from test fixtures #14075
Remove mr from test fixtures #14075
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.
Change looks good to me!
Same tests failed as https://github.com/rapidsai/cudf/actions/runs/6126279847/job/16630671974?pr=14057
|
e2da4d3
to
9a5ebb9
Compare
/ok to test |
@robertmaynard and I have both spent some time trying to investigate the issues here. We were unable to reproduce on at least one other arm node, and Robert said he was not able to reproduce this using CUDA 12. Since the issue appears to be very specific and compiler-related, I'm going to close this PR for now. The plan will be to reopen this once we drop CUDA 11 support since the issue appears to be specific to CUDA 11. |
Tracking in #14236 |
Description
This is a follow-up to #14057. The PR was split in two because in that PR this particular change led to an unexpected and very specific test failure (see #14057 (comment)). If this PR reproduces that issue we will need to debug more closely on the appropriate hardware.
Checklist