-
Notifications
You must be signed in to change notification settings - Fork 3
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
Include confidence_threshold in submit_image_query params #286
Conversation
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.
LGTM
spec/public-api.yaml
Outdated
@@ -1367,8 +1375,7 @@ components: | |||
- ALGORITHM | |||
count: | |||
type: integer | |||
minimum: null |
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.
I made a fix earlier in the week, these are just plainly invalid swagger that is corrected
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.
Incidentally, to whom it may concern, this is a great website
https://editor.swagger.io/
Pass
confidence_threshold
through to thesubmit_image_query
call, which will fix some edge behavior and improve overall consistency. Will merge after the backend change is deployed.