Skip to content

Fix classification filtering#866

Merged
czaloom merged 2 commits intomainfrom
czaloom-fix-clf-annotation-filter-1-30-26
Feb 2, 2026
Merged

Fix classification filtering#866
czaloom merged 2 commits intomainfrom
czaloom-fix-clf-annotation-filter-1-30-26

Conversation

@czaloom
Copy link
Copy Markdown
Collaborator

@czaloom czaloom commented Jan 30, 2026

Issue

  • valor_lite.classification.Evaluator.filter classmethod did not correctly remove rows containing filtered fields

Solution

  • Fixed the valid mask to omit any partially filtered rows
  • Add unit tests to all CV task types

@czaloom czaloom requested a review from a team as a code owner January 30, 2026 21:52
@czaloom czaloom self-assigned this Feb 2, 2026
@czaloom czaloom merged commit 67832a9 into main Feb 2, 2026
4 checks passed
@czaloom czaloom deleted the czaloom-fix-clf-annotation-filter-1-30-26 branch February 2, 2026 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants