Skip to content

Commit

Permalink
Fix put trained model spec (#2169)
Browse files Browse the repository at this point in the history
Built in models like ELSER do not require an inference_config be
specified. #2098 adjusted the spec to reflect this in one place,
but not in the put trained model API spec. This PR completes the
change.
  • Loading branch information
droberts195 authored and github-actions[bot] committed Jul 5, 2023
1 parent 993c7cd commit 1caa19d
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
6 changes: 3 additions & 3 deletions output/schema/schema.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion output/typescript/types.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -65,9 +65,10 @@ export interface Request extends RequestBase {
/**
* The default configuration for inference. This can be either a regression
* or classification configuration. It must match the underlying
* definition.trained_model's target_type.
* definition.trained_model's target_type. For pre-packaged models such as
* ELSER the config is not required.
*/
inference_config: InferenceConfigCreateContainer
inference_config?: InferenceConfigCreateContainer
/**
* The input field names for the model definition.
*/
Expand Down

0 comments on commit 1caa19d

Please sign in to comment.