Remove deprecated response_type parameter from query settings
- Bump API version to 0254 - Remove response format UI controls - Hard-code response_type in query params - Add migration for version 19 - Clean up settings store structure
This commit is contained in:
parent
abeaac84fa
commit
93a3e47134
4 changed files with 9 additions and 44 deletions
|
|
@ -1 +1 @@
|
||||||
__api_version__ = "0253"
|
__api_version__ = "0254"
|
||||||
|
|
|
||||||
|
|
@ -40,7 +40,6 @@ export default function QuerySettings() {
|
||||||
// Default values for reset functionality
|
// Default values for reset functionality
|
||||||
const defaultValues = useMemo(() => ({
|
const defaultValues = useMemo(() => ({
|
||||||
mode: 'mix' as QueryMode,
|
mode: 'mix' as QueryMode,
|
||||||
response_type: 'Multiple Paragraphs',
|
|
||||||
top_k: 40,
|
top_k: 40,
|
||||||
chunk_top_k: 20,
|
chunk_top_k: 20,
|
||||||
max_entity_tokens: 6000,
|
max_entity_tokens: 6000,
|
||||||
|
|
@ -153,46 +152,6 @@ export default function QuerySettings() {
|
||||||
</div>
|
</div>
|
||||||
</>
|
</>
|
||||||
|
|
||||||
{/* Response Format */}
|
|
||||||
<>
|
|
||||||
<TooltipProvider>
|
|
||||||
<Tooltip>
|
|
||||||
<TooltipTrigger asChild>
|
|
||||||
<label htmlFor="response_format_select" className="ml-1 cursor-help">
|
|
||||||
{t('retrievePanel.querySettings.responseFormat')}
|
|
||||||
</label>
|
|
||||||
</TooltipTrigger>
|
|
||||||
<TooltipContent side="left">
|
|
||||||
<p>{t('retrievePanel.querySettings.responseFormatTooltip')}</p>
|
|
||||||
</TooltipContent>
|
|
||||||
</Tooltip>
|
|
||||||
</TooltipProvider>
|
|
||||||
<div className="flex items-center gap-1">
|
|
||||||
<Select
|
|
||||||
value={querySettings.response_type}
|
|
||||||
onValueChange={(v) => handleChange('response_type', v)}
|
|
||||||
>
|
|
||||||
<SelectTrigger
|
|
||||||
id="response_format_select"
|
|
||||||
className="hover:bg-primary/5 h-9 cursor-pointer focus:ring-0 focus:ring-offset-0 focus:outline-0 active:right-0 flex-1 text-left [&>span]:break-all [&>span]:line-clamp-1"
|
|
||||||
>
|
|
||||||
<SelectValue />
|
|
||||||
</SelectTrigger>
|
|
||||||
<SelectContent>
|
|
||||||
<SelectGroup>
|
|
||||||
<SelectItem value="Multiple Paragraphs">{t('retrievePanel.querySettings.responseFormatOptions.multipleParagraphs')}</SelectItem>
|
|
||||||
<SelectItem value="Single Paragraph">{t('retrievePanel.querySettings.responseFormatOptions.singleParagraph')}</SelectItem>
|
|
||||||
<SelectItem value="Bullet Points">{t('retrievePanel.querySettings.responseFormatOptions.bulletPoints')}</SelectItem>
|
|
||||||
</SelectGroup>
|
|
||||||
</SelectContent>
|
|
||||||
</Select>
|
|
||||||
<ResetButton
|
|
||||||
onClick={() => handleReset('response_type')}
|
|
||||||
title="Reset to default (Multiple Paragraphs)"
|
|
||||||
/>
|
|
||||||
</div>
|
|
||||||
</>
|
|
||||||
|
|
||||||
{/* Top K */}
|
{/* Top K */}
|
||||||
<>
|
<>
|
||||||
<TooltipProvider>
|
<TooltipProvider>
|
||||||
|
|
|
||||||
|
|
@ -357,6 +357,7 @@ export default function RetrievalTesting() {
|
||||||
const queryParams = {
|
const queryParams = {
|
||||||
...state.querySettings,
|
...state.querySettings,
|
||||||
query: actualQuery,
|
query: actualQuery,
|
||||||
|
response_type: 'Multiple Paragraphs',
|
||||||
conversation_history: effectiveHistoryTurns > 0
|
conversation_history: effectiveHistoryTurns > 0
|
||||||
? prevMessages
|
? prevMessages
|
||||||
.filter((m) => m.isError !== true)
|
.filter((m) => m.isError !== true)
|
||||||
|
|
|
||||||
|
|
@ -123,7 +123,6 @@ const useSettingsStoreBase = create<SettingsState>()(
|
||||||
|
|
||||||
querySettings: {
|
querySettings: {
|
||||||
mode: 'global',
|
mode: 'global',
|
||||||
response_type: 'Multiple Paragraphs',
|
|
||||||
top_k: 40,
|
top_k: 40,
|
||||||
chunk_top_k: 20,
|
chunk_top_k: 20,
|
||||||
max_entity_tokens: 6000,
|
max_entity_tokens: 6000,
|
||||||
|
|
@ -239,7 +238,7 @@ const useSettingsStoreBase = create<SettingsState>()(
|
||||||
{
|
{
|
||||||
name: 'settings-storage',
|
name: 'settings-storage',
|
||||||
storage: createJSONStorage(() => localStorage),
|
storage: createJSONStorage(() => localStorage),
|
||||||
version: 18,
|
version: 19,
|
||||||
migrate: (state: any, version: number) => {
|
migrate: (state: any, version: number) => {
|
||||||
if (version < 2) {
|
if (version < 2) {
|
||||||
state.showEdgeLabel = false
|
state.showEdgeLabel = false
|
||||||
|
|
@ -336,6 +335,12 @@ const useSettingsStoreBase = create<SettingsState>()(
|
||||||
// Add userPromptHistory field for older versions
|
// Add userPromptHistory field for older versions
|
||||||
state.userPromptHistory = []
|
state.userPromptHistory = []
|
||||||
}
|
}
|
||||||
|
if (version < 19) {
|
||||||
|
// Remove deprecated response_type parameter
|
||||||
|
if (state.querySettings) {
|
||||||
|
delete state.querySettings.response_type
|
||||||
|
}
|
||||||
|
}
|
||||||
return state
|
return state
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue