fix: wrong setting

pull/19840/head
crazywoola 11 months ago
parent 8bdee2512e
commit 111458f80e

@ -170,12 +170,6 @@ const ConfigPopup: FC<PopupProps> = ({
const configuredProviderPanel = () => { const configuredProviderPanel = () => {
const configuredPanels: JSX.Element[] = [] const configuredPanels: JSX.Element[] = []
if (arizeConfig)
configuredPanels.push(arizePanel)
if (phoenixConfig)
configuredPanels.push(phoenixPanel)
if (langFuseConfig) if (langFuseConfig)
configuredPanels.push(langfusePanel) configuredPanels.push(langfusePanel)
@ -188,6 +182,12 @@ const ConfigPopup: FC<PopupProps> = ({
if (weaveConfig) if (weaveConfig)
configuredPanels.push(weavePanel) configuredPanels.push(weavePanel)
if (arizeConfig)
configuredPanels.push(arizePanel)
if (phoenixConfig)
configuredPanels.push(phoenixPanel)
return configuredPanels return configuredPanels
} }

@ -62,20 +62,15 @@ const Panel: FC = () => {
} }
const inUseTracingProvider: TracingProvider | null = tracingStatus?.tracing_provider || null const inUseTracingProvider: TracingProvider | null = tracingStatus?.tracing_provider || null
const InUseProviderIcon const providerIconMap: Record<TracingProvider, React.FC<{ className?: string }>> = {
= inUseTracingProvider === TracingProvider.arize [TracingProvider.arize]: ArizeIcon,
? ArizeIcon [TracingProvider.phoenix]: PhoenixIcon,
: inUseTracingProvider === TracingProvider.phoenix [TracingProvider.langSmith]: LangsmithIcon,
? PhoenixIcon [TracingProvider.langfuse]: LangfuseIcon,
: inUseTracingProvider === TracingProvider.langSmith [TracingProvider.opik]: OpikIcon,
? LangsmithIcon [TracingProvider.weave]: WeaveIcon,
: inUseTracingProvider === TracingProvider.langfuse }
? LangfuseIcon const InUseProviderIcon = inUseTracingProvider ? providerIconMap[inUseTracingProvider] : undefined
: inUseTracingProvider === TracingProvider.opik
? OpikIcon
: inUseTracingProvider === TracingProvider.weave
? WeaveIcon
: LangsmithIcon
const [arizeConfig, setArizeConfig] = useState<ArizeConfig | null>(null) const [arizeConfig, setArizeConfig] = useState<ArizeConfig | null>(null)
const [phoenixConfig, setPhoenixConfig] = useState<PhoenixConfig | null>(null) const [phoenixConfig, setPhoenixConfig] = useState<PhoenixConfig | null>(null)
@ -83,15 +78,15 @@ const Panel: FC = () => {
const [langFuseConfig, setLangFuseConfig] = useState<LangFuseConfig | null>(null) const [langFuseConfig, setLangFuseConfig] = useState<LangFuseConfig | null>(null)
const [opikConfig, setOpikConfig] = useState<OpikConfig | null>(null) const [opikConfig, setOpikConfig] = useState<OpikConfig | null>(null)
const [weaveConfig, setWeaveConfig] = useState<WeaveConfig | null>(null) const [weaveConfig, setWeaveConfig] = useState<WeaveConfig | null>(null)
const hasConfiguredTracing = !!(langSmithConfig || langFuseConfig || opikConfig || weaveConfig) const hasConfiguredTracing = !!(langSmithConfig || langFuseConfig || opikConfig || weaveConfig || arizeConfig || phoenixConfig)
const fetchTracingConfig = async () => { const fetchTracingConfig = async () => {
const { tracing_config: arizeConfig, has_not_configured: arizeHasNotConfig } = await doFetchTracingConfig({ appId, provider: TracingProvider.arize }) const { tracing_config: arizeConfig, has_not_configured: arizeHasNotConfig } = await doFetchTracingConfig({ appId, provider: TracingProvider.arize })
if (!arizeHasNotConfig) if (!arizeHasNotConfig)
setLangSmithConfig(arizeConfig as ArizeConfig) setArizeConfig(arizeConfig as ArizeConfig)
const { tracing_config: phoenixConfig, has_not_configured: phoenixHasNotConfig } = await doFetchTracingConfig({ appId, provider: TracingProvider.phoenix }) const { tracing_config: phoenixConfig, has_not_configured: phoenixHasNotConfig } = await doFetchTracingConfig({ appId, provider: TracingProvider.phoenix })
if (!phoenixHasNotConfig) if (!phoenixHasNotConfig)
setLangSmithConfig(phoenixConfig as PhoenixConfig) setPhoenixConfig(phoenixConfig as PhoenixConfig)
const { tracing_config: langSmithConfig, has_not_configured: langSmithHasNotConfig } = await doFetchTracingConfig({ appId, provider: TracingProvider.langSmith }) const { tracing_config: langSmithConfig, has_not_configured: langSmithHasNotConfig } = await doFetchTracingConfig({ appId, provider: TracingProvider.langSmith })
if (!langSmithHasNotConfig) if (!langSmithHasNotConfig)
setLangSmithConfig(langSmithConfig as LangSmithConfig) setLangSmithConfig(langSmithConfig as LangSmithConfig)

Loading…
Cancel
Save