Skip to content

Conversation

@xzrderek
Copy link
Contributor

@xzrderek xzrderek commented Jan 14, 2026

Note

Introduces consistent model ID normalization for Fireworks when routing via LiteLLM and applies it across rollout processors.

  • Adds normalize_fireworks_model_for_litellm in pytest/utils.py; removes old normalize_fireworks_model from evaluation_test_utils.py
  • Applies normalization and sets row.input_metadata.completion_params (and config.completion_params where used) in default_agent_rollout_processor.py, default_single_turn_rollout_process.py, default_mcp_gym_rollout_processor.py, default_klavis_sandbox_rollout_processor.py, openenv_rollout_processor.py, remote_rollout_processor.py, and github_action_rollout_processor.py
  • Ensures policies and requests use normalized model/params; minor style/typing cleanups (e.g., pyright ignores)

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

@xzrderek xzrderek merged commit 8349922 into main Jan 14, 2026
14 of 17 checks passed
@xzrderek xzrderek deleted the derekx/normalize-in-rollout-processors branch January 14, 2026 01:24
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