fix: 4209 - Inconsistent max tokens value persistence

This commit is contained in:
Louis 2024-12-04 12:31:40 +07:00
parent 9e634c783f
commit fe190658c1
No known key found for this signature in database
GPG Key ID: 44FA9F4D33C37DE2
2 changed files with 23 additions and 12 deletions

View File

@ -187,15 +187,19 @@ const ModelDropdown = ({
],
})
const overriddenSettings =
model?.settings.ctx_len && model.settings.ctx_len > 4096
? { ctx_len: 4096 }
: {}
const defaultContextLength = Math.min(
8192,
model?.settings.ctx_len ?? 8192
)
const overriddenParameters = {
ctx_len: Math.min(8192, model?.settings.ctx_len ?? 8192),
max_tokens: defaultContextLength,
}
const modelParams = {
...model?.parameters,
...model?.settings,
...overriddenSettings,
...overriddenParameters,
}
// Update model parameter to the thread state

View File

@ -105,14 +105,21 @@ export const useCreateNewThread = () => {
enabled: true,
settings: assistant.tools && assistant.tools[0].settings,
}
const overriddenSettings =
defaultModel?.settings.ctx_len && defaultModel.settings.ctx_len > 2048
? { ctx_len: 4096 }
: {}
const overriddenParameters = defaultModel?.parameters.max_tokens
? { max_tokens: 4096 }
: {}
// Default context length is 8192
const defaultContextLength = Math.min(
8192,
defaultModel?.settings.ctx_len ?? 8192
)
const overriddenSettings = {
ctx_len: defaultContextLength,
}
// Use ctx length by default
const overriddenParameters = {
max_tokens: defaultContextLength,
}
const createdAt = Date.now()
let instructions: string | undefined = assistant.instructions