Skip to content

fix(openai-compatible): support custom temperature setting (#60)#174

Open
zerone0x wants to merge 1 commit into
plmbr:mainfrom
zerone0x:fix/60-openai-compatible-temperature
Open

fix(openai-compatible): support custom temperature setting (#60)#174
zerone0x wants to merge 1 commit into
plmbr:mainfrom
zerone0x:fix/60-openai-compatible-temperature

Conversation

@zerone0x
Copy link
Copy Markdown
Contributor

@zerone0x zerone0x commented May 1, 2026

Summary

  • add an optional temperature provider property for OpenAI-compatible chat and inline completion models
  • only pass temperature through when the field is set, preserving current provider defaults otherwise
  • add regression tests for parsing and request forwarding

Testing

  • .venv/bin/pytest -q tests/test_openai_compatible_llm_provider.py

Closes #60

Copy link
Copy Markdown
Collaborator

@mbektas mbektas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@mbektas
Copy link
Copy Markdown
Collaborator

mbektas commented May 5, 2026

@zerone0x can you resolve the conflicts

@pjdoland pjdoland added the enhancement New feature or request label May 5, 2026
@mbektas mbektas added the blocked Blocked due to conflicts or no response from author label May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked Blocked due to conflicts or no response from author enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support Customizable Temperature in OpenAI-Compatible API

3 participants