fix: use EXTENDED_CONTEXT_STEP for slider step value in settings (#5989)

Co-authored-by: George Zhao <georgezhao@SKJLAB>
This commit is contained in:
George Zhao 2025-05-14 20:08:47 +08:00 committed by GitHub
parent 577d180e1d
commit 79f66f0d50
3 changed files with 11 additions and 4 deletions

View File

@ -1,6 +1,7 @@
export const DEFAULT_TEMPERATURE = 1.0
export const DEFAULT_CONTEXTCOUNT = 5
export const EXTENDED_CONTEXT_LIMIT = 100
export const EXTENDED_CONTEXT_STEP = 1
export const DEFAULT_MAX_TOKENS = 4096
export const DEFAULT_KNOWLEDGE_DOCUMENT_COUNT = 6
export const DEFAULT_KNOWLEDGE_THRESHOLD = 0.0

View File

@ -6,6 +6,7 @@ import {
DEFAULT_MAX_TOKENS,
DEFAULT_TEMPERATURE,
EXTENDED_CONTEXT_LIMIT,
EXTENDED_CONTEXT_STEP,
isMac,
isWindows
} from '@renderer/config/constant'
@ -219,7 +220,7 @@ const SettingsTab: FC<Props> = (props) => {
onChange={setContextCount}
onChangeComplete={onContextCountChange}
value={validAndChangeContextCount(contextCount, enableMaxContexts, EXTENDED_CONTEXT_LIMIT)}
step={!enableMaxContexts ? 1 : 10}
step={!enableMaxContexts ? 1 : EXTENDED_CONTEXT_STEP}
tooltip={{ formatter: formatSliderTooltip }}
/>
</Col>

View File

@ -2,7 +2,12 @@ import { DeleteOutlined, PlusOutlined, QuestionCircleOutlined } from '@ant-desig
import ModelAvatar from '@renderer/components/Avatar/ModelAvatar'
import { HStack } from '@renderer/components/Layout'
import SelectModelPopup from '@renderer/components/Popups/SelectModelPopup'
import { DEFAULT_CONTEXTCOUNT, DEFAULT_TEMPERATURE, EXTENDED_CONTEXT_LIMIT } from '@renderer/config/constant'
import {
DEFAULT_CONTEXTCOUNT,
DEFAULT_TEMPERATURE,
EXTENDED_CONTEXT_LIMIT,
EXTENDED_CONTEXT_STEP
} from '@renderer/config/constant'
import { SettingRow } from '@renderer/pages/settings'
import { EVENT_NAMES, EventEmitter } from '@renderer/services/EventService'
import { Assistant, AssistantSettingCustomParameters, AssistantSettings } from '@renderer/types'
@ -310,7 +315,7 @@ const AssistantModelSettings: FC<Props> = ({ assistant, updateAssistant, updateA
onChange={setContextCount}
onChangeComplete={onContextCountChange}
value={validAndChangeContextCount(contextCount, enableMaxContexts, EXTENDED_CONTEXT_LIMIT)}
step={!enableMaxContexts ? 1 : 10}
step={!enableMaxContexts ? 1 : EXTENDED_CONTEXT_STEP}
tooltip={{ formatter: formatSliderTooltip }}
/>
</Col>
@ -318,7 +323,7 @@ const AssistantModelSettings: FC<Props> = ({ assistant, updateAssistant, updateA
<InputNumber
min={0}
max={!enableMaxContexts ? 10 : EXTENDED_CONTEXT_LIMIT}
step={!enableMaxContexts ? 1 : 10}
step={!enableMaxContexts ? 1 : EXTENDED_CONTEXT_STEP}
value={contextCount}
changeOnBlur
onChange={(value) => {