Skip to content

auto no prefix needed#404

Merged
xzrderek merged 3 commits intomainfrom
derekx/fireworks_ai_no_prefix
Jan 8, 2026
Merged

auto no prefix needed#404
xzrderek merged 3 commits intomainfrom
derekx/fireworks_ai_no_prefix

Conversation

@xzrderek
Copy link
Contributor

@xzrderek xzrderek commented Jan 8, 2026

Note

Introduces automatic normalization of Fireworks model identifiers to ensure compatibility with LiteLLM and downstream components.

  • Adds normalize_fireworks_model in evaluation_test_utils and applies it in evaluation_test to both global completion_params and per-invocation kwargs
  • Updates tests to pass unprefixed Fireworks models (e.g., accounts/.../models/...); Pydantic agent factory strips the prefix when constructing the provider-specific model; remote server test continues to assert prefixed model in row.input_metadata.completion_params
  • Minor cleanup: remove unused imports, simplify use_priority_scheduler condition, small formatting tweaks

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

@xzrderek xzrderek merged commit ed22857 into main Jan 8, 2026
2 of 3 checks passed
@xzrderek xzrderek deleted the derekx/fireworks_ai_no_prefix branch January 8, 2026 21:39
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