-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add version for ML Predict, Train APIs #763
base: main
Are you sure you want to change the base?
Add version for ML Predict, Train APIs #763
Conversation
Signed-off-by: Nathalie Jonathan <[email protected]>
Changes AnalysisCommit SHA: 5a22c2e API ChangesSummary
ReportThe full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/12604469711/artifacts/2384568173 API Coverage
|
Signed-off-by: Daniel (dB.) Doubrovkine <[email protected]>
Hello! |
I tried to set up a 1.3 OpenSearch cluster and run the test, and it seems that the tests for train, predict, train and predict APIs passed. |
Great, I apologize for my excessive OCD. I was also trying OS 1.3 with Docker Compose, which we have in the ML folder, and it wouldn't start on these versions, so I thought there might be an error here. Most likely, in the future, if we introduce 1.3 and test on them, we will have to redo the Docker Compose we have there now |
No problem at all! I I referenced the Docker Compose from here, which potentially more accurate :) |
Description
Added version for ML train, predict, train and predict APIs.
Issues Resolved
Part of opensearch-project/opensearch-py#867.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.