Skip to content

fix(enterprise): persist llm model temperature in create and update APIs#253

Open
zfaustk wants to merge 3 commits intodataelement:mainfrom
zfaustk:fix/enterprise-llm-model-persistence
Open

fix(enterprise): persist llm model temperature in create and update APIs#253
zfaustk wants to merge 3 commits intodataelement:mainfrom
zfaustk:fix/enterprise-llm-model-persistence

Conversation

@zfaustk
Copy link
Copy Markdown

@zfaustk zfaustk commented Apr 1, 2026

Summary

Fixes #247 by persisting temperature in both enterprise LLM model write paths:

  • store temperature when creating /api/enterprise/llm-models
  • store temperature when updating /api/enterprise/llm-models/{model_id}
  • add focused regression coverage for both code paths

Validation

  • pytest tests/test_enterprise_llm_model_api.py

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.

Bug: LLM Model temperature field ignored in create/update API

1 participant