Skip to content

Commit 192c2b8

Browse files
Fix [Batch inference] The model_path input only supports store paths (#2012)
1 parent dbfdef1 commit 192c2b8

File tree

3 files changed

+39
-7
lines changed

3 files changed

+39
-7
lines changed

src/common/TargetPath/targetPath.util.js

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -155,37 +155,44 @@ export const getTargetPathOptions = hiddenOptionsIds => [
155155
{
156156
className: 'path-type-v3io',
157157
label: 'V3IO',
158-
id: V3IO_INPUT_PATH_SCHEME
158+
id: V3IO_INPUT_PATH_SCHEME,
159+
hidden: hiddenOptionsIds?.includes(V3IO_INPUT_PATH_SCHEME)
159160
},
160161
{
161162
className: 'path-type-s3',
162163
label: 'S3',
163-
id: S3_INPUT_PATH_SCHEME
164+
id: S3_INPUT_PATH_SCHEME,
165+
hidden: hiddenOptionsIds?.includes(S3_INPUT_PATH_SCHEME)
164166
},
165167
{
166168
className: 'path-type-http',
167169
label: 'HTTP',
168-
id: HTTP_STORAGE_INPUT_PATH_SCHEME
170+
id: HTTP_STORAGE_INPUT_PATH_SCHEME,
171+
hidden: hiddenOptionsIds?.includes(HTTP_STORAGE_INPUT_PATH_SCHEME)
169172
},
170173
{
171174
className: 'path-type-https',
172175
label: 'HTTPS',
173-
id: HTTPS_STORAGE_INPUT_PATH_SCHEME
176+
id: HTTPS_STORAGE_INPUT_PATH_SCHEME,
177+
hidden: hiddenOptionsIds?.includes(HTTPS_STORAGE_INPUT_PATH_SCHEME)
174178
},
175179
{
176180
className: 'path-type-az',
177181
label: 'Azure storage',
178-
id: AZURE_STORAGE_INPUT_PATH_SCHEME
182+
id: AZURE_STORAGE_INPUT_PATH_SCHEME,
183+
hidden: hiddenOptionsIds?.includes(AZURE_STORAGE_INPUT_PATH_SCHEME)
179184
},
180185
{
181186
className: 'path-type-gs',
182187
label: 'Google storage',
183-
id: GOOGLE_STORAGE_INPUT_PATH_SCHEME
188+
id: GOOGLE_STORAGE_INPUT_PATH_SCHEME,
189+
hidden: hiddenOptionsIds?.includes(GOOGLE_STORAGE_INPUT_PATH_SCHEME)
184190
},
185191
{
186192
className: 'path-type-dbfs',
187193
label: 'Databricks filesystem',
188-
id: DBFS_STORAGE_INPUT_PATH_SCHEME
194+
id: DBFS_STORAGE_INPUT_PATH_SCHEME,
195+
hidden: hiddenOptionsIds?.includes(DBFS_STORAGE_INPUT_PATH_SCHEME)
189196
}
190197
]
191198

src/constants.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,8 @@ export const FULL_VIEW_MODE = 'full'
5353

5454
export const REQUEST_CANCELED = 'Request canceled'
5555

56+
export const MODEL_PATH_DATA_INPUT = 'model_path'
57+
5658
/*=========== PAGES & TABS =============*/
5759

5860
export const PROJECTS_PAGE = 'PROJECTS'

src/elements/FormDataInputsTable/FormDataInputsRow/FormDataInputsRow.js

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,16 @@ import {
3131
handleStoreInputPathChange,
3232
targetPathInitialState
3333
} from '../../../common/TargetPath/targetPath.util'
34+
import {
35+
AZURE_STORAGE_INPUT_PATH_SCHEME,
36+
DBFS_STORAGE_INPUT_PATH_SCHEME,
37+
GOOGLE_STORAGE_INPUT_PATH_SCHEME,
38+
HTTP_STORAGE_INPUT_PATH_SCHEME,
39+
HTTPS_STORAGE_INPUT_PATH_SCHEME,
40+
MODEL_PATH_DATA_INPUT,
41+
S3_INPUT_PATH_SCHEME,
42+
V3IO_INPUT_PATH_SCHEME
43+
} from '../../../constants'
3444

3545
const FormDataInputsRow = ({
3646
applyChanges,
@@ -86,6 +96,19 @@ const FormDataInputsRow = ({
8696
density="normal"
8797
formState={formState}
8898
formStateFieldInfo={`${rowPath}.data.fieldInfo`}
99+
hiddenSelectOptionsIds={
100+
fieldData.data.name === MODEL_PATH_DATA_INPUT && fieldData.isPredefined
101+
? [
102+
V3IO_INPUT_PATH_SCHEME,
103+
S3_INPUT_PATH_SCHEME,
104+
HTTP_STORAGE_INPUT_PATH_SCHEME,
105+
HTTPS_STORAGE_INPUT_PATH_SCHEME,
106+
AZURE_STORAGE_INPUT_PATH_SCHEME,
107+
GOOGLE_STORAGE_INPUT_PATH_SCHEME,
108+
DBFS_STORAGE_INPUT_PATH_SCHEME
109+
]
110+
: []
111+
}
89112
inputDefaultValue={editingItem.data.fieldInfo?.value}
90113
name={`${rowPath}.data.path`}
91114
required

0 commit comments

Comments
 (0)