Problem/Motivation
Currently maxResults values for Label Detection is hardcoded, making impossible for a user to customize this value based on a per-project basis. It would be nice if this could be exposed to a configuration value.
Proposed resolution
Expose this maxResults value for Label Detection used in the API calls to be configured by the site-builder.
Remaining tasks
None
User interface changes
None
API changes
None
Data model changes
None
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | interdiff-9-10.txt | 15.08 KB | ajalan065 |
| #10 | 2783121-10.patch | 7.3 KB | ajalan065 |
| #9 | expose-max-results-to-config-2783121-9.patch | 11.35 KB | marcoscano |
| #9 | interdiff.txt | 521 bytes | marcoscano |
| #6 | expose-max-results-to-config-2783121-6.patch | 11.35 KB | marcoscano |
Comments
Comment #2
marcoscanoComment #4
naveenvalechaThanks for the patch!
change form key to 'max_results'
don't append the max_results before with every form key.
Make it simple 'label_detection'
Address at other places as well.
Comment #5
marcoscanoComment #6
marcoscanoAddressed feedback in #4
Thanks!
Comment #7
ajalan065 commentedComment #9
marcoscanoSorry there was a typo in the schema definition, now fixed
Comment #10
ajalan065 commentedHi,
I would like to focus light on some points:
1. Only the Label Detection feature of the Google Vision API should be configurable. These are related to taxonomy tagging, hence, I agree to the point that it should be in the hands of the user.
However, other features need not be configurable, as it would be of no use. Here is why?
1. Landmark, Logo, Optical Character Detection have been used to fill the Alt Text attribute of the image file. So, IMHO, its of no use to leave it on the end user.
2. Safe Search detection feature is used to detect and avoid any explicit contents in images being uploaded. Hence, no use of making it configurable to have many values. It does not actually need any manipulation on the hands of the user.
3. Same is the case of User Emotion Detection.
4. As far as Image Properties is concerned, the image would have only one dominant color, either Red, Green or Blue or in the worst case, all the three (if the image is black and white).
We have hard coded the 'maxResults' with those values, as they would suffice our demand and requirements to use the features.
Here, I am uploading the new patch with necessary changes to reflect the configuration. Also, I have added and updated the tests to verify whether the value is configurable or not.
Comment #11
ajalan065 commentedComment #12
naveenvalechaleaving this open for the thoughts of eugene on this, what he thinks about #10
Comment #13
eugene.ilyin commentedI prefer the solution of Arpit. His explanation in #10 sounds pretty logical.
Comment #15
naveenvalechaComment #16
eugene.ilyin commentedCommitted.