Skip to content

Remove arbitrary score threshold for the test_qn test with softmax#6636

Merged
rapids-bot[bot] merged 2 commits intorapidsai:branch-25.06from
csadorf:mitigate-issue-6633
May 7, 2025
Merged

Remove arbitrary score threshold for the test_qn test with softmax#6636
rapids-bot[bot] merged 2 commits intorapidsai:branch-25.06from
csadorf:mitigate-issue-6633

Conversation

@csadorf
Copy link
Copy Markdown
Contributor

@csadorf csadorf commented May 7, 2025

Resolves #6633.

@csadorf csadorf requested a review from a team as a code owner May 7, 2025 17:20
@csadorf csadorf requested review from bdice and teju85 May 7, 2025 17:20
@github-actions github-actions Bot added the Cython / Python Cython or Python issue label May 7, 2025
@csadorf csadorf added bug Something isn't working tests Unit testing for project non-breaking Non-breaking change and removed Cython / Python Cython or Python issue labels May 7, 2025
dantegd
dantegd previously requested changes May 7, 2025
Comment thread python/cuml/cuml/tests/test_qn.py Outdated
@github-actions github-actions Bot added the Cython / Python Cython or Python issue label May 7, 2025
@csadorf csadorf requested a review from dantegd May 7, 2025 17:24
@csadorf csadorf changed the title Lower the score threshold for the test_qn test with softmax. Remove arbitrary score threshold for the test_qn test with softmax May 7, 2025
@csadorf csadorf self-assigned this May 7, 2025
@csadorf csadorf dismissed dantegd’s stale review May 7, 2025 17:52

Requested change is applied.

@csadorf
Copy link
Copy Markdown
Contributor Author

csadorf commented May 7, 2025

/merge

@rapids-bot rapids-bot Bot merged commit e136b07 into rapidsai:branch-25.06 May 7, 2025
74 checks passed
@csadorf csadorf deleted the mitigate-issue-6633 branch May 7, 2025 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Cython / Python Cython or Python issue non-breaking Non-breaking change tests Unit testing for project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

QN Test Failure with Softmax Loss Function

3 participants