fix: clear instances when page hidden
This commit is contained in:
@@ -66,7 +66,6 @@ interface ModelsProps {
|
||||
handleCategoryChange: (val: any) => void;
|
||||
onViewLogs: () => void;
|
||||
onCancelViewLogs: () => void;
|
||||
allInstances: ModelInstanceListItem[];
|
||||
queryParams: {
|
||||
page: number;
|
||||
perPage: number;
|
||||
@@ -120,7 +119,6 @@ const Models: React.FC<ModelsProps> = ({
|
||||
onViewLogs,
|
||||
onCancelViewLogs,
|
||||
handleCategoryChange,
|
||||
allInstances,
|
||||
deleteIds,
|
||||
dataSource,
|
||||
gpuDeviceList,
|
||||
@@ -144,7 +142,6 @@ const Models: React.FC<ModelsProps> = ({
|
||||
defaultSortOrder: 'descend'
|
||||
});
|
||||
|
||||
const updateRef = useRef(false);
|
||||
const [openLogModal, setOpenLogModal] = useState(false);
|
||||
const [openAddModal, setOpenAddModal] = useState(false);
|
||||
const [openDeployModal, setOpenDeployModal] = useState<any>({
|
||||
@@ -673,24 +670,17 @@ const Models: React.FC<ModelsProps> = ({
|
||||
|
||||
const renderChildren = useCallback(
|
||||
(list: any, parent?: any) => {
|
||||
let childList = list;
|
||||
if (allInstances.length && !updateRef.current) {
|
||||
childList = list.filter((item: any) => {
|
||||
return allInstances.some((instance) => instance.id === item.id);
|
||||
});
|
||||
updateRef.current = true;
|
||||
}
|
||||
return (
|
||||
<InstanceItem
|
||||
list={childList}
|
||||
list={list}
|
||||
modelData={parent}
|
||||
gpuDeviceList={gpuDeviceList}
|
||||
gpuDeviceList={[]}
|
||||
workerList={workerList}
|
||||
handleChildSelect={handleChildSelect}
|
||||
></InstanceItem>
|
||||
);
|
||||
},
|
||||
[workerList, allInstances]
|
||||
[workerList]
|
||||
);
|
||||
|
||||
const generateSource = useCallback((record: ListItem) => {
|
||||
@@ -746,20 +736,6 @@ const Models: React.FC<ModelsProps> = ({
|
||||
}
|
||||
};
|
||||
|
||||
useEffect(() => {
|
||||
const handleVisibilityChange = async () => {
|
||||
if (document.visibilityState === 'hidden') {
|
||||
updateRef.current = false;
|
||||
}
|
||||
};
|
||||
|
||||
document.addEventListener('visibilitychange', handleVisibilityChange);
|
||||
|
||||
return () => {
|
||||
document.removeEventListener('visibilitychange', handleVisibilityChange);
|
||||
};
|
||||
}, []);
|
||||
|
||||
return (
|
||||
<>
|
||||
<PageContainer
|
||||
|
||||
@@ -61,6 +61,7 @@ const sourceOptions = [
|
||||
{
|
||||
label: 'models.form.localPath',
|
||||
value: modelSourceMap.local_path_value,
|
||||
locale: true,
|
||||
key: 'local_path'
|
||||
}
|
||||
];
|
||||
|
||||
@@ -1,7 +1,5 @@
|
||||
import TableContext from '@/components/seal-table/table-context';
|
||||
import useSetChunkRequest, {
|
||||
createAxiosToken as generateAxiosToken
|
||||
} from '@/hooks/use-chunk-request';
|
||||
import useSetChunkRequest from '@/hooks/use-chunk-request';
|
||||
import useUpdateChunkedList from '@/hooks/use-update-chunk-list';
|
||||
import { queryWorkersList } from '@/pages/resources/apis';
|
||||
import {
|
||||
@@ -11,17 +9,10 @@ import {
|
||||
import _ from 'lodash';
|
||||
import qs from 'query-string';
|
||||
import { memo, useCallback, useEffect, useRef, useState } from 'react';
|
||||
import {
|
||||
MODELS_API,
|
||||
MODEL_INSTANCE_API,
|
||||
queryModelsInstances,
|
||||
queryModelsList
|
||||
} from './apis';
|
||||
import { MODELS_API, MODEL_INSTANCE_API, queryModelsList } from './apis';
|
||||
import TableList from './components/table-list';
|
||||
import { ListItem } from './config/types';
|
||||
|
||||
const INSTANCE_SYNC = 'instance_sync';
|
||||
|
||||
const Models: React.FC = () => {
|
||||
const { setChunkRequest, createAxiosToken } = useSetChunkRequest();
|
||||
const { setChunkRequest: setModelInstanceChunkRequest } =
|
||||
@@ -39,7 +30,6 @@ const Models: React.FC = () => {
|
||||
total: 0
|
||||
});
|
||||
|
||||
const [allInstances, setAllInstances] = useState<any[]>([]);
|
||||
const [gpuDeviceList, setGpuDeviceList] = useState<GPUDeviceItem[]>([]);
|
||||
const [workerList, setWorkerList] = useState<WokerListItem[]>([]);
|
||||
const chunkRequedtRef = useRef<any>();
|
||||
@@ -69,24 +59,6 @@ const Models: React.FC = () => {
|
||||
}
|
||||
});
|
||||
|
||||
const fetchModelsInstances = async () => {
|
||||
try {
|
||||
instancesToken.current?.cancel?.();
|
||||
instancesToken.current = generateAxiosToken();
|
||||
const params = {
|
||||
page: 1,
|
||||
perPage: 100
|
||||
};
|
||||
const data: any = await queryModelsInstances(params, {
|
||||
token: instancesToken.current?.token
|
||||
});
|
||||
setAllInstances(data.items || []);
|
||||
} catch (error) {
|
||||
// ignore
|
||||
setAllInstances([]);
|
||||
}
|
||||
};
|
||||
|
||||
const getWorkerList = async () => {
|
||||
try {
|
||||
const data = await queryWorkersList({ page: 1, perPage: 100 });
|
||||
@@ -249,7 +221,7 @@ const Models: React.FC = () => {
|
||||
const handleVisibilityChange = async () => {
|
||||
if (document.visibilityState === 'visible') {
|
||||
isPageHidden.current = false;
|
||||
await fetchModelsInstances();
|
||||
// await fetchModelsInstances();
|
||||
await Promise.all([
|
||||
createModelsChunkRequest(),
|
||||
createModelsInstanceChunkRequest()
|
||||
@@ -284,7 +256,6 @@ const Models: React.FC = () => {
|
||||
handlePageChange={handlePageChange}
|
||||
handleDeleteSuccess={fetchData}
|
||||
onViewLogs={handleOnViewLogs}
|
||||
allInstances={allInstances}
|
||||
onCancelViewLogs={handleOnCancelViewLogs}
|
||||
queryParams={queryParams}
|
||||
loading={dataSource.loading}
|
||||
|
||||
Reference in New Issue
Block a user