Skip to content

take out output dataloader#396

Merged
xzrderek merged 5 commits intomainfrom
derekx/take-out-outputdataloader
Jan 7, 2026
Merged

take out output dataloader#396
xzrderek merged 5 commits intomainfrom
derekx/take-out-outputdataloader

Conversation

@xzrderek
Copy link
Contributor

@xzrderek xzrderek commented Jan 6, 2026

Note

Removes support for custom output data loaders and standardizes trace fetching via Fireworks.

  • Drop output_data_loader from GithubActionRolloutProcessor and RemoteRolloutProcessor; always use default_fireworks_output_data_loader
  • Update tests to monkeypatch/wrap the default loader to track rollout_ids; remove bespoke loaders; deprecate test_remote_langfuse
  • Remote server: sanitize messages into OpenAI-compatible dicts (exclude None) and use that payload in completions
  • Minor doc/example updates (TypeScript server README) and small assertions/cleanups in tests

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

@xzrderek xzrderek merged commit da7fc9d into main Jan 7, 2026
17 of 18 checks passed
@xzrderek xzrderek deleted the derekx/take-out-outputdataloader branch January 7, 2026 00:48
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