Skip to content

Commit 4f5dc66

Browse files
committed
LLMs: Params: fix temperature slider becoming a range when unlocked at effort
1 parent 9500706 commit 4f5dc66

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

src/modules/llms/models-modal/LLMParametersEditor.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -363,8 +363,11 @@ export function LLMParametersEditor(props: {
363363
step={0.1}
364364
defaultValue={0.5 /* FIXME: this wasn't FALLBACK_LLM_PARAM_TEMPERATURE, but we shall not need this */}
365365
valueLabelDisplay={props.parameters?.llmTemperature === undefined || antThinkingEnabled ? 'auto' : 'on'} // detect user-overridden or not
366-
value={llmTemperature ?? (overheat ? [1, 1] : [0.5, 0.5]) /* null and undefined both would become undefined (uncontrolled) in the slider */}
367-
onChange={value => onChangeParameter({ llmTemperature: value })}
366+
value={llmTemperature ?? ( // nullish: single thumb when interactive (unlocked), collapsed two-thumb 'empty' look only when disabled - an array on a live slider becomes a range control
367+
oaiTempUnlockedByNoReasoning ? 0.5 // default when unlocked by OpenAI + effort 'none'
368+
: overheat ? [1, 1] : [0.5, 0.5]
369+
)}
370+
onChange={value => onChangeParameter({ llmTemperature: Array.isArray(value) ? value[0] : value })}
368371
endAdornment={
369372
<Tooltip arrow disableInteractive title={overheat ? 'Disable LLM Overheating' : 'Increase Max LLM Temperature to 2'} sx={{ p: 1 }}>
370373
<IconButton

0 commit comments

Comments
 (0)