-
Notifications
You must be signed in to change notification settings - Fork 61
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
Rename APIs for model serving framework #159
Changes from 8 commits
2385d06
5c769d5
5b78d46
10aed7b
e808ce5
146e8ab
20a8fab
9470d1e
dafeef6
75a765a
f0e4b80
86e7c51
3a7d72b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
Deploy Model | ||
================== | ||
|
||
.. currentmodule:: opensearch_py_ml | ||
|
||
.. autofunction:: opensearch_py_ml.ml_commons.MLCommonClient.deploy_model |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
Register Model | ||
================== | ||
|
||
.. currentmodule:: opensearch_py_ml | ||
|
||
.. autofunction:: opensearch_py_ml.ml_commons.MLCommonClient.register_model |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
Register Pretrained Model | ||
================== | ||
|
||
.. currentmodule:: opensearch_py_ml | ||
|
||
.. autofunction:: opensearch_py_ml.ml_commons.MLCommonClient.register_pretrained_model |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
Undeploy Model | ||
================== | ||
|
||
.. currentmodule:: opensearch_py_ml | ||
|
||
.. autofunction:: opensearch_py_ml.ml_commons.MLCommonClient.undeploy_model |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,20 +12,41 @@ Upload Model | |
|
||
api/ml_commons_upload_api | ||
|
||
Register Model | ||
~~~~~~~~~~~~~~ | ||
.. toctree:: | ||
:maxdepth: 2 | ||
|
||
api/ml_commons_register_api | ||
|
||
Upload Pretrained Model | ||
~~~~~~~~~~~~~~~~~~~~~~~ | ||
.. toctree:: | ||
:maxdepth: 2 | ||
|
||
api/ml_commons_upload_pretrained_api | ||
|
||
Register Pretrained Model | ||
~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
.. toctree:: | ||
:maxdepth: 2 | ||
|
||
api/ml_commons_register_pretrained_api | ||
|
||
Load Model | ||
~~~~~~~~~~ | ||
.. toctree:: | ||
:maxdepth: 2 | ||
|
||
api/ml_commons_load_api | ||
|
||
Deploy Model | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same here. |
||
~~~~~~~~~~ | ||
.. toctree:: | ||
:maxdepth: 2 | ||
|
||
api/ml_commons_deploy_api | ||
|
||
Get Task Info | ||
~~~~~~~~~~~~~ | ||
.. toctree:: | ||
|
@@ -54,6 +75,13 @@ Unload Model | |
|
||
api/ml_commons_unload_model_api | ||
|
||
Undeploy Model | ||
~~~~~~~~~~~~ | ||
.. toctree:: | ||
:maxdepth: 2 | ||
|
||
api/ml_commons_undeploy_model_api | ||
|
||
Delete Model | ||
~~~~~~~~~~~~ | ||
.. toctree:: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,7 +6,7 @@ | |
# GitHub history for details. | ||
|
||
ML_BASE_URI = "/_plugins/_ml" | ||
MODEL_UPLOAD_CHUNK_SIZE = 10_000_000 | ||
MODEL_REGISTER_CHUNK_SIZE = 10_000_000 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. How about change to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Or to match the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done |
||
MODEL_MAX_SIZE = 4_000_000_000 | ||
BUF_SIZE = 65536 # lets read stuff in 64kb chunks! | ||
TIMEOUT = 120 # timeout for synchronous method calls in seconds |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to cover ====== all the way to Model.