-
Notifications
You must be signed in to change notification settings - Fork 69
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
fix Auto-merge: Branch 23.08 merge 23.06 #482
fix Auto-merge: Branch 23.08 merge 23.06 #482
Conversation
This PR: - fixes a couple of tests which were failing due to recent changes, leading to aggregate results being unsorted - resolves docs build failures, which were caused by datashader rendering issues with latest numpy(1.24.3), fixed by adding a pin for numpy (<1.24, similar to the version pins in cugraph). @ajschmidt8, could you check if adding this version pin is fine? Authors: - Ajay Thorve (https://github.com/AjayThorve) Approvers: - Allan (https://github.com/exactlyallan) - Ray Douglass (https://github.com/raydouglass) URL: rapidsai#479
Align with the rest of RAPIDS on these requirements. Also needed for CUDA 12 support. Authors: - https://github.com/jakirkham - Ajay Thorve (https://github.com/AjayThorve) Approvers: - Ray Douglass (https://github.com/raydouglass) - Ajay Thorve (https://github.com/AjayThorve) URL: rapidsai#480
…e every week (rapidsai#478) Based on the discussion in rapidsai#452, adding a weekly schedule to run the external dependency testing workflow. cc @exactlyallan @ajschmidt8 Authors: - Ajay Thorve (https://github.com/AjayThorve) Approvers: - AJ Schmidt (https://github.com/ajschmidt8) URL: rapidsai#478
19ff4e1
to
30f54a7
Compare
30f54a7
to
31cf579
Compare
@AjayThorve, it looks like a merge conflict turned up. Can you rebase to pull in the latest changes? |
31cf579
to
f27b2c5
Compare
done |
@AjayThorve, I don't think that worked as intended. This PR should have one more commit than the number of commits in the conflicted PR (#481). Since that PR has 3 commits, this PR should have those commits plus one more commit for the merge conflicts (so 4 total commits). |
f27b2c5
to
5a46979
Compare
5a46979
to
c17e0d0
Compare
Manually resolve conflicts in the auto-merge PR #481