[4.1.7.168] Regression: lead images not positioned correctly
Closed, ResolvedPublic

Description

Steps to reproduce

  1. Clean install
  2. Go to Barack Obama

Expected results

Actual results

'Nellie Kim' (lead image; the thumbnail for Saved/Recent is correct)


Dev notes

The property where focal rects were stored was initialized to an empty array, meaning that we always thought face detection previously ran, even when it didn't. Property is no initialized to nil as expected, and tests were added to ensure proper behavior in the following cases:

  • New image w/o any data on disk (detection hasn't run, no faces)
  • Image w/ data on disk w/o a focalRects field (detection hasn't run, no faces—same as above)
  • Image w/ data on disk with focalRects field set to empty array (detection has run, no faces)
  • Image w/ data on disk w/ focalRects field set to non-empty array (detection has run, hasFaces is true)
Etonkovidova updated the task description. (Show Details)
Etonkovidova raised the priority of this task from to Needs Triage.
Etonkovidova added a subscriber: Etonkovidova.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 27 2015, 11:39 PM
BGerstle-WMF set Security to None.
BGerstle-WMF claimed this task.
BGerstle-WMF renamed this task from [4.1.7.170 alpha] Regression: lead images not positioned correctly to [4.1.7.168] Regression: lead images not positioned correctly.
BGerstle-WMF updated the task description. (Show Details)
BGerstle-WMF updated the task description. (Show Details)

Checked on 4.1.7.171 beta(release candidate) on iPad mini iOS 8.2 - 'Barack Obama' display is correct.
'Nellie Kim'(and also 'Viceregal consort of Canada') lead img are still displayed incorrectly, but it's not a regression. Closing the issue.

Etonkovidova updated the task description. (Show Details)Jul 30 2015, 7:55 PM
Etonkovidova closed this task as Resolved.