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

Update client to auto-choose endpoint #400

Closed
milesgranger opened this issue Aug 12, 2019 · 0 comments · Fixed by #425
Closed

Update client to auto-choose endpoint #400

milesgranger opened this issue Aug 12, 2019 · 0 comments · Fixed by #425

Comments

@milesgranger
Copy link
Contributor

milesgranger commented Aug 12, 2019

Problem

Model is more-or-less hard coded to /anomaly/prediction

Context

#368 sets the return code from /anomaly/prediction as 422 Unprocessable Entity if the model is not capable of doing anomaly detection.

Potential solution(s)

Should now be able to set the client to always try /anomaly/prediction first and then switch endpoints to /prediction should the return code be 422

Otherwise we'll need to update the model's metadata to reflect what type of model it is and then decide in the client which endpoint to connect to based on that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant