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

Allow multi-model repositories for Tensorflow Serving #3206

Closed
ukclivecox opened this issue May 17, 2021 · 0 comments · Fixed by #3220
Closed

Allow multi-model repositories for Tensorflow Serving #3206

ukclivecox opened this issue May 17, 2021 · 0 comments · Fixed by #3220

Comments

@ukclivecox
Copy link
Contributor

Check if we can relax the constraint that the model path needs to exist when using Tensorflow protocol

if r.Protocol == api.ProtocolTensorflow {
if modelName != "" {
if graphNode = v1.GetPredictiveUnit(&r.predictor.Graph, modelName); graphNode == nil {
r.respondWithError(w, nil, fmt.Errorf("Failed to find model %s", modelName))
return
}

@ukclivecox ukclivecox added the triage Needs to be triaged and prioritised accordingly label May 17, 2021
@ukclivecox ukclivecox added this to Triage in Backlog via automation May 17, 2021
@ukclivecox ukclivecox moved this from Triage to Next Sprint in Backlog May 17, 2021
@ukclivecox ukclivecox self-assigned this May 19, 2021
@ukclivecox ukclivecox removed this from Next Sprint in Backlog May 19, 2021
@ukclivecox ukclivecox added this to To do in MLOps Sprint 8 (Ending 29/05/2020) via automation May 19, 2021
@ukclivecox ukclivecox moved this from To do to In progress in MLOps Sprint 8 (Ending 29/05/2020) May 19, 2021
@ukclivecox ukclivecox removed the triage Needs to be triaged and prioritised accordingly label May 20, 2021
@ukclivecox ukclivecox moved this from In progress to Review in progress in MLOps Sprint 8 (Ending 29/05/2020) May 21, 2021
MLOps Sprint 8 (Ending 29/05/2020) automation moved this from Review in progress to Done May 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant