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.

Sign up for GitHub

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

Jump to bottom

Fix kmeans pytest to correctly compute fp output error #5426

Merged
merged 2 commits into from
May 18, 2023
Merged

Fix kmeans pytest to correctly compute fp output error #5426

merged 2 commits into from
May 18, 2023

Conversation

mdoijade
Copy link
Contributor

-- This PR is on top build fix - #5425
-- fixes the fp error checking logic and set the tolerance to account for 3xTF32 based fusedL2NN kernel

@mdoijade mdoijade requested review from a team as code owners May 18, 2023 14:15
@github-actions github-actions bot added CUDA/C++ Cython / Python Cython or Python issue labels May 18, 2023
@mdoijade mdoijade added bug Something isn't working non-breaking Non-breaking change labels May 18, 2023
@cjnolet
Copy link
Member

cjnolet commented May 18, 2023

/merge

@rapids-bot rapids-bot bot merged commit dd432c6 into rapidsai:branch-23.06 May 18, 2023
@hcho3 hcho3 mentioned this pull request May 25, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dantegd dantegd dantegd approved these changes

Assignees
No one assigned
Labels
bug Something isn't working CUDA/C++ Cython / Python Cython or Python issue non-breaking Non-breaking change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mdoijade @cjnolet @dantegd @hcho3