This crop to Mitch McConnell's (turtle) face is not an expectd lead image, since it is only a fraction of the entire "wanted" image.
iOS 10.2.1 beta
Build 1046
| Josve05a | |
| Jan 25 2017, 2:59 AM |
| F6035226: turtle.gif | |
| Mar 3 2017, 12:15 AM |
| F6034712: T156217 Part 1.PNG | |
| Mar 3 2017, 12:09 AM |
| F6034710: T156217 Part 2.PNG | |
| Mar 3 2017, 12:09 AM |
| F5353789: IMG_4703.PNG | |
| Jan 25 2017, 2:59 AM |
| F5353794: IMG_4704.PNG | |
| Jan 25 2017, 2:59 AM |
This crop to Mitch McConnell's (turtle) face is not an expectd lead image, since it is only a fraction of the entire "wanted" image.
iOS 10.2.1 beta
Build 1046
This is just a simple code change just have it make sure its cropped to the chest-area or like a mug shot type crop perhaps?
@AMroczkowski this is the face detection issue I mentioned at standup.
Per this and the other ticket I think, face detection or no, we should limit the amount of scaling we do with lead images. To solve both, lets limit the scaling of lead images to 2x and then only top align if we do scale.
Lets try that and see a few test cases.
iirc the face detection method gives an array of the faces found with rects. if we compare the area of the largest (1st?) rect we can compare it to the area of the image for this logic
In practice. 25% seemed too high. I cut it down to 5% and it actually seems to before better in practice.
Testing on iPhone 5s (iOS 10.2.1) and Wikipedia app 5.4.0 (1081). According to the below screencaps this is fixed now as the correct lead image is displayed in the article: