Skip to content

respect dataset adapter#439

Merged
xzrderek merged 1 commit intomainfrom
derekx/ep-jsonl-respect-dataset-adapter
Mar 23, 2026
Merged

respect dataset adapter#439
xzrderek merged 1 commit intomainfrom
derekx/ep-jsonl-respect-dataset-adapter

Conversation

@xzrderek
Copy link
Contributor

@xzrderek xzrderek commented Mar 23, 2026

Note

Medium Risk
Changes how JSONL datasets are loaded when EP_JSONL_PATH is set by optionally routing rows through dataset_adapter, which could alter row construction and labeling for existing env-driven workflows.

Overview
When EP_JSONL_PATH is used to auto-create a DynamicDataLoader, JSONL lines are now loaded as raw dicts and passed through the configured dataset_adapter (instead of always constructing EvaluationRow(**dict)).

evaluation_test now forwards its dataset_adapter into parse_ep_dataloaders, preserving adapter-specific fields (e.g., custom labels) for env-driven JSONL runs while keeping legacy behavior when no adapter is provided.

Written by Cursor Bugbot for commit b42717b. This will update automatically on new commits. Configure here.

@xzrderek xzrderek merged commit b0cbc25 into main Mar 23, 2026
1 of 2 checks passed
@xzrderek xzrderek deleted the derekx/ep-jsonl-respect-dataset-adapter branch March 23, 2026 06:53
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