Skip to content

Commit 26d0e20

Browse files
Fix put trained model spec (#2169) (#2170)
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. Co-authored-by: David Roberts <[email protected]>
1 parent 993c7cd commit 26d0e20

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

output/schema/schema.json

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

output/typescript/types.ts

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

specification/ml/put_trained_model/MlPutTrainedModelRequest.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,9 +65,10 @@ export interface Request extends RequestBase {
6565
/**
6666
* The default configuration for inference. This can be either a regression
6767
* or classification configuration. It must match the underlying
68-
* definition.trained_model's target_type.
68+
* definition.trained_model's target_type. For pre-packaged models such as
69+
* ELSER the config is not required.
6970
*/
70-
inference_config: InferenceConfigCreateContainer
71+
inference_config?: InferenceConfigCreateContainer
7172
/**
7273
* The input field names for the model definition.
7374
*/

0 commit comments

Comments
 (0)