agentjido / req_llm / ef60c770cd524e7c730619c39e721a0f3c1ea018-PR-11

23%

main: 49%

Build Type

Pull #11

github

Committed by mikehostetler

Commit Message

Fix OpenAI o1 model parameter translation

- Add translate_options/3 callback to Provider behavior for model-specific parameter translation
- Implement OpenAI o1 model parameter translation: max_tokens → max_completion_tokens and drop temperature
- Fix encode_chat_body to use maybe_put for conditional parameters instead of hardcoding
- Add comprehensive tests for translation functionality and integration
- Update documentation to reflect new options translation capability
- Add demo script proving Issue #8 resolution

Resolves #8: OpenAI o1 models now work with standard ReqLLM parameters through automatic translation

Amp-Thread-ID: https://ampcode.com/threads/T-f9ddd7ac-ace0-4db3-86fb-9aa00e33592a
Co-authored-by: Amp <amp@ampcode.com>

Pull Request Pull Request #11: Fix OpenAI o1 model parameter translation (resolves #8)

Run Details

3 of 24 new or added lines in 2 files covered. (12.5%)

1 existing line in 1 file now uncovered.

508 of 2401 relevant lines covered (21.16%)

233.92 hits per line