fix range search

Signed-off-by: shanghaikid <jiangruiyi@gmail.com>

client/src/components/customSnackBar/CustomSnackBar.tsx

@@ -19,7 +19,8 @@ const useStyles = makeStyles((theme: Theme) =>
   createStyles({
     root: {
       borderRadius: '4px',
-      maxWidth: '300px',
+      maxWidth: '50vh',
+      wordBreak: 'break-all'
     },
     topRight: {
       [theme.breakpoints.up('md')]: {
@@ -37,7 +38,7 @@ const CustomSnackBar: FC<CustomSnackBarType> = props => {
     vertical,
     horizontal,
     open,
-    autoHideDuration = 2000,
+    autoHideDuration = 2500,
     type,
     message,
     onClose,

client/src/consts/Milvus.ts

@@ -67,7 +67,7 @@ export type searchKeywordsType =
   | 'round_decimal'
   | 'level'
   | 'search_list'
-  | 'range'
+  | 'radius'
   | 'range_filter';
 
 export type indexConfigType = {
@@ -211,7 +211,7 @@ export const DEFAULT_METRIC_VALUE_MAP = {
 
 // search params default value map
 export const DEFAULT_SEARCH_PARAM_VALUE_MAP: {
-  [key in searchKeywordsType]: number;
+  [key in searchKeywordsType]?: number;
 } = {
   // range: [top_k, 32768]
   ef: 250,
@@ -221,11 +221,8 @@ export const DEFAULT_SEARCH_PARAM_VALUE_MAP: {
   search_k: 250,
   // range: [10, 300]
   search_length: 10,
-  round_decimal: -1,
   level: 1,
   search_list: 20,
-  range: 20,
-  range_filter: 10
 };
 
 export const DEFAULT_NLIST_VALUE = 1024;

client/src/context/Root.tsx

@@ -61,7 +61,7 @@ export const RootProvider = (props: { children: React.ReactNode }) => {
     message: '',
     vertical: 'top',
     horizontal: 'right',
-    autoHideDuration: 3000,
+    autoHideDuration: 1000,
   });
   const [dialog, setDialog] = useState<DialogType>(DefaultDialogConfigs);
   const [drawer, setDrawer]: any = useState({

client/src/pages/search/SearchParams.tsx

@@ -68,7 +68,7 @@ const SearchParams: FC<SearchParamsProps> = ({
     }
 
     const commonParams: searchKeywordsType[] = [
-      'range',
+      'radius',
       'range_filter',
       'round_decimal',
     ];
@@ -78,8 +78,14 @@ const SearchParams: FC<SearchParamsProps> = ({
   }, [indexType, openSnackBar, warningTrans]);
 
   const handleInputChange = useCallback(
-    (key: string, value: number) => {
-      const form = { ...searchParamsForm, [key]: value };
+    (key: string, value: number | string) => {
+      let form = { ...searchParamsForm };
+      if (value === '') {
+        delete form[key];
+      } else {
+        form = { ...searchParamsForm, [key]: Number(value) };
+      }
+
       handleFormChange(form);
     },
     [handleFormChange, searchParamsForm]
@@ -98,6 +104,7 @@ const SearchParams: FC<SearchParamsProps> = ({
         value,
         handleChange,
         isInt = true,
+        required = true,
       } = params;
 
       // search_k range is special compared to others，need to be handled separately
@@ -114,22 +121,13 @@ const SearchParams: FC<SearchParamsProps> = ({
         variant: 'filled',
         type: 'number',
         value,
-        validations: [
-          {
-            rule: 'require',
-            errorText: warningTrans('required', { name: label }),
-          },
-        ],
+        validations: [],
       };
-      if (!isSearchK && min && max) {
+
+      if (required) {
         config.validations?.push({
-          rule: 'range',
-          errorText: warningTrans('range', { min, max }),
-          extraParam: {
-            min,
-            max,
-            type: 'number',
-          },
+          rule: 'require',
+          errorText: warningTrans('required', { name: label }),
         });
       }
 
@@ -174,14 +172,15 @@ const SearchParams: FC<SearchParamsProps> = ({
         [key in searchKeywordsType]: SearchParamInputConfig;
       } = {
         round_decimal: {
-          label: 'Round',
+          label: 'round',
           key: 'round_decimal',
           value: searchParamsForm['round_decimal'] || '',
           min: -1,
           max: 10,
           isInt: true,
+          required: false,
           handleChange: value => {
-            handleInputChange('round_decimal', Number(value));
+            handleInputChange('round_decimal', value);
           },
           className: classes.inlineInput,
         },
@@ -193,31 +192,33 @@ const SearchParams: FC<SearchParamsProps> = ({
           max: nlist,
           isInt: true,
           handleChange: value => {
-            handleInputChange('nprobe', Number(value));
+            handleInputChange('nprobe', value);
           },
           className: classes.inlineInput,
         },
-        range: {
-          label: 'range',
-          key: 'range',
-          value: searchParamsForm['range'] || '',
+        radius: {
+          label: 'radius',
+          key: 'radius',
+          value: searchParamsForm['radius'] || '',
           min: 1,
-          max: 500,
+          max: 1024,
           isInt: false,
+          required: false,
           handleChange: value => {
-            handleInputChange('range', Number(value));
+            handleInputChange('radius', value);
           },
           className: classes.inlineInput,
         },
         range_filter: {
-          label: 'range_filter',
+          label: 'range filter',
           key: 'range_filter',
           value: searchParamsForm['range_filter'] || '',
           min: 1,
-          max: 500,
+          max: 1024,
           isInt: false,
+          required: false,
           handleChange: value => {
-            handleInputChange('range_filter', Number(value));
+            handleInputChange('range_filter', value);
           },
           className: classes.inlineInput,
         },
@@ -229,7 +230,7 @@ const SearchParams: FC<SearchParamsProps> = ({
           max: 32768,
           isInt: true,
           handleChange: value => {
-            handleInputChange('ef', Number(value));
+            handleInputChange('ef', value);
           },
         },
         level: {
@@ -240,7 +241,7 @@ const SearchParams: FC<SearchParamsProps> = ({
           max: 3,
           isInt: true,
           handleChange: value => {
-            handleInputChange('level', Number(value));
+            handleInputChange('level', value);
           },
         },
         search_k: {
@@ -252,7 +253,7 @@ const SearchParams: FC<SearchParamsProps> = ({
           max: Infinity,
           isInt: true,
           handleChange: value => {
-            handleInputChange('search_k', Number(value));
+            handleInputChange('search_k', value);
           },
         },
         search_length: {
@@ -263,7 +264,7 @@ const SearchParams: FC<SearchParamsProps> = ({
           max: 300,
           isInt: true,
           handleChange: value => {
-            handleInputChange('search_length', Number(value));
+            handleInputChange('search_length', value);
           },
         },
         search_list: {
@@ -274,7 +275,7 @@ const SearchParams: FC<SearchParamsProps> = ({
           max: 65535,
           isInt: true,
           handleChange: value => {
-            handleInputChange('search_list', Number(value));
+            handleInputChange('search_list', value);
           },
         },
       };

client/src/pages/search/Types.ts

@@ -52,6 +52,7 @@ export interface SearchParamInputConfig {
   value: number | string;
   handleChange: (value: number) => void;
   className?: string;
+  required?: boolean;
 }
 
 export interface VectorSearchParam {

client/src/pages/search/VectorSearch.tsx

@@ -3,12 +3,7 @@ import { Typography, Button, Card, CardContent } from '@material-ui/core';
 import { useTranslation } from 'react-i18next';
 import { useLocation } from 'react-router-dom';
 import { ALL_ROUTER_TYPES } from '@/router/Types';
-import {
-  useNavigationHook,
-  useSearchResult,
-  usePaginationHook,
-  useTimeTravelHook,
-} from '@/hooks';
+import { useNavigationHook, useSearchResult, usePaginationHook } from '@/hooks';
 import { dataContext } from '@/context';
 import CustomSelector from '@/components/customSelector/CustomSelector';
 import { ColDefinitionsType } from '@/components/grid/Types';
@@ -92,9 +87,6 @@ const VectorSearch = () => {
     handleGridSort,
   } = usePaginationHook(searchResultMemo || []);
 
-  const { timeTravel, setTimeTravel, timeTravelInfo, handleDateTimeChange } =
-    useTimeTravelHook();
-
   const collectionOptions: Option[] = useMemo(
     () =>
       collections.map(c => ({
@@ -317,7 +309,6 @@ const VectorSearch = () => {
     setSearchResult(null);
     setFilterFields([]);
     setExpression('');
-    setTimeTravel(null);
   };
 
   const handleSearch = async (topK: number, expr = expression) => {
@@ -337,7 +328,6 @@ const VectorSearch = () => {
       search_params: searchParamPairs,
       vectors: [parseValue(vectors)],
       vector_type: fieldType,
-      travel_timestamp: timeTravelInfo.timestamp,
     };
 
     setTableLoading(true);