Skip to content

Conversation

@yuyutaotao
Copy link
Collaborator

Motivation

  • Expose per-intent sampling temperature so callers can control model sampling via environment or agent config.
  • Preserve deterministic behavior for vlm-ui-tars by defaulting its temperature to 0.0 unless overridden.
  • Improve observability by surfacing the resolved temperature in profile statistics logs.

Description

  • Add environment keys and constants for MIDSCENE_MODEL_TEMPERATURE, MIDSCENE_INSIGHT_MODEL_TEMPERATURE, and MIDSCENE_PLANNING_MODEL_TEMPERATURE and wire them into MODEL_ENV_KEYS and related config structures.
  • Extend environment types and IModelConfig with a temperature field and parse it in parseOpenaiSdkConfig to include temperature in the resolved model config.
  • Use the resolved modelConfig.temperature in callAI (falling back to 0.0 for vlm-ui-tars) and include it in commonConfig sent to the SDK.
  • Append temperature to the comma-separated debugProfileStats output in both streaming and non-streaming code paths, and update English/Chinese model-config docs to document the new env variables.

Testing

  • No automated tests were run for this change.

Codex Task

@netlify
Copy link

netlify bot commented Dec 24, 2025

Deploy Preview for midscene ready!

Name Link
🔨 Latest commit b4a674d
🔍 Latest deploy log https://app.netlify.com/projects/midscene/deploys/694bb8d6d4f82e0008bc412a
😎 Deploy Preview https://deploy-preview-1654--midscene.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@yuyutaotao yuyutaotao changed the title feat(ai): add model temperature config and log temperature in profile stats feat(ai): set the default temperature to 0 and allow users to configure it Dec 25, 2025
@yuyutaotao yuyutaotao merged commit 683af18 into main Dec 25, 2025
11 checks passed
@yuyutaotao yuyutaotao deleted the codex/add-model-temperature-parameters branch December 25, 2025 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants