agentjido / req_llm / e8f08de9a9667270bc40a535ed5371944976da70

49%

Build Type

push

github

Committed by web-flow

Commit Message

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

* Implement provider option translation and enhance error handling

- Introduced `translate_options/3` callback in the `ReqLLM.Provider` module to allow providers to modify option keys and values before API calls, addressing parameter name differences and model-specific restrictions.
- Added `on_unsupported` option in the schema to manage unsupported parameter translations with configurable behaviors (warn, error, ignore).
- Enhanced the `ReqLLM.Generation` module to utilize the new translation capabilities, ensuring proper handling of options during request preparation.
- Implemented translation helper functions in the `ReqLLM.Provider.DSL` module for renaming and dropping options, along with combining warnings.
- Added comprehensive test coverage for translation functionality in both the `OpenAI` provider and the generation pipeline, ensuring robust validation of option handling.

* Enhance documentation for provider-specific parameter translation

- Updated AGENTS.md to include the `translate_options/3` callback, detailing its role in handling model-specific parameter requirements.
- Added a section in usage-rules.md explaining the automatic parameter translation feature, with examples demonstrating its implementation.
- Revised README.md to highlight the automatic parameter translation for model-specific requirements, improving clarity for users.

* 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... (continued)

Run Details

3 of 32 new or added lines in 2 files covered. (9.38%)

1 existing line in 1 file now uncovered.

532 of 2307 relevant lines covered (23.06%)

243.67 hits per line