Skip to content
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

Some RBC3D fixes #530

Merged
merged 41 commits into from
May 12, 2022
Merged

Conversation

cjnolet
Copy link
Member

@cjnolet cjnolet commented Feb 26, 2022

This PR fixes an issue where the query size was still assumed to be the index size in a couple places.

@cjnolet cjnolet requested review from a team as code owners February 26, 2022 01:35
@github-actions github-actions bot removed the CMake label Feb 26, 2022
@cjnolet cjnolet added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Feb 26, 2022
cjnolet added 3 commits March 1, 2022 11:43
 Please enter the commit message for your changes. Lines starting
@cjnolet cjnolet changed the title Fea 2204 rbc 3d 2 Some RBC3D fixes Mar 1, 2022
@cjnolet cjnolet self-assigned this Mar 1, 2022

// TODO: There seem to be discrepancies here only when
// the entire test suite is executed.
// Ref: https://github.com/rapidsai/raft/issues/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing exact issue number?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ha! Thanks for noticing this.

@cjnolet cjnolet changed the base branch from branch-22.04 to branch-22.06 March 28, 2022 16:55
@github-actions
Copy link

This PR has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates. This PR will be labeled inactive-90d if there is no activity in the next 60 days.

@cjnolet
Copy link
Member Author

cjnolet commented May 12, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit ef625e8 into rapidsai:branch-22.06 May 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cpp improvement Improvement / enhancement to an existing function inactive-30d non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants