Skip to content

feat: add DatasetValidator for testset structural and content validation - #52

Merged
zurfjereluhmie merged 2 commits into
mainfrom
feat/testset-validation
May 7, 2026
Merged

feat: add DatasetValidator for testset structural and content validation#52
zurfjereluhmie merged 2 commits into
mainfrom
feat/testset-validation

Conversation

@zurfjereluhmie

Copy link
Copy Markdown
Owner

Add DatasetValidator class with ValidationResult and ValidationIssue dataclasses.
Checks required columns, empty tags, and empty expected_output values.
Reports row-level issues with line numbers. 14 unit tests cover all validation paths.

Add DatasetValidator class with ValidationResult and ValidationIssue dataclasses.
Checks required columns, empty tags, and empty expected_output values.
Reports row-level issues with line numbers. 14 unit tests cover all validation paths.
Add DatasetValidator integration to the CLI:
- New 'evalwire validate --csv PATH' command reports structural and row-level issues
- New '--strict' flag on 'evalwire upload' aborts the upload when validation fails
- 7 CLI tests covering valid/invalid CSV, custom column names, and strict mode
@zurfjereluhmie
zurfjereluhmie force-pushed the feat/testset-validation branch from 927d2aa to 8f70592 Compare May 7, 2026 15:54
@zurfjereluhmie
zurfjereluhmie merged commit cb7374e into main May 7, 2026
10 checks passed
@zurfjereluhmie
zurfjereluhmie deleted the feat/testset-validation branch May 7, 2026 15:56
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.

1 participant