Skip to content
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

[Enhancement] Register Model API return model_id and Deploy Model API return deployment results #1146

Closed
Zhangxunmt opened this issue Jul 19, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@Zhangxunmt
Copy link
Collaborator

Zhangxunmt commented Jul 19, 2023

In remote inference, we reuse the same API workflow in the Register and Deploy APIs. These 2 APIs both only return task_id in the response. Users have to use Get Task API to obtain the results for the Register and Deploy model Results.

This workflow can be improved that 1) register model_id included in the Register_Model API response and 2) Deployment status included in the Deploy_Model API response.

So, after the improvement. The overall workflow will be:

  1. Create a connector.
  2. Register a model. (Get Task not needed anymore)
  3. Deploy the model. (Get Task not needed anymore)
  4. Predict the model.
@Zhangxunmt Zhangxunmt added bug Something isn't working enhancement New feature or request labels Jul 19, 2023
@Zhangxunmt Zhangxunmt self-assigned this Jul 19, 2023
@ylwu-amzn ylwu-amzn removed the bug Something isn't working label Aug 2, 2023
@Zhangxunmt Zhangxunmt changed the title [Improvement] Register Model API return model_id and Deploy Model API return deployment results [Enhancement] Register Model API return model_id and Deploy Model API return deployment results Aug 25, 2023
@ylwu-amzn ylwu-amzn moved this to In Progress in ml-commons projects Aug 30, 2023
@Zhangxunmt
Copy link
Collaborator Author

#1329
#1336

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Released
Development

No branches or pull requests

2 participants