From 4041fd7e5cee0cbac0f97d65743a5f2bd30e30f4 Mon Sep 17 00:00:00 2001 From: JzoNg Date: Wed, 29 Apr 2026 20:10:37 +0800 Subject: [PATCH] fix(web): auto select model in evaluation --- .../components/judge-model-selector.tsx | 20 +------------------ .../components/layout/pipeline-evaluation.tsx | 1 - 2 files changed, 1 insertion(+), 20 deletions(-) diff --git a/web/app/components/evaluation/components/judge-model-selector.tsx b/web/app/components/evaluation/components/judge-model-selector.tsx index 8f9ee4aff6..ac954ed086 100644 --- a/web/app/components/evaluation/components/judge-model-selector.tsx +++ b/web/app/components/evaluation/components/judge-model-selector.tsx @@ -1,39 +1,21 @@ 'use client' import type { EvaluationResourceProps } from '../types' -import { useEffect } from 'react' import { ModelTypeEnum } from '@/app/components/header/account-setting/model-provider-page/declarations' import { useModelList } from '@/app/components/header/account-setting/model-provider-page/hooks' import ModelSelector from '@/app/components/header/account-setting/model-provider-page/model-selector' import { useEvaluationResource, useEvaluationStore } from '../store' import { decodeModelSelection, encodeModelSelection } from '../utils' -type JudgeModelSelectorProps = EvaluationResourceProps & { - autoSelectFirst?: boolean -} - const JudgeModelSelector = ({ resourceType, resourceId, - autoSelectFirst = true, -}: JudgeModelSelectorProps) => { +}: EvaluationResourceProps) => { const { data: modelList } = useModelList(ModelTypeEnum.textGeneration) const resource = useEvaluationResource(resourceType, resourceId) const setJudgeModel = useEvaluationStore(state => state.setJudgeModel) const selectedModel = decodeModelSelection(resource.judgeModelId) - useEffect(() => { - if (!autoSelectFirst || resource.judgeModelId || !modelList.length) - return - - const firstProvider = modelList[0] - const firstModel = firstProvider.models[0] - if (!firstProvider || !firstModel) - return - - setJudgeModel(resourceType, resourceId, encodeModelSelection(firstProvider.provider, firstModel.model)) - }, [autoSelectFirst, modelList, resource.judgeModelId, resourceId, resourceType, setJudgeModel]) - return (