Page MenuHomePhabricator

[SPIKE 2 hrs] Performance: Facial detection speed-ups
Closed, ResolvedPublic

Description

Outcome: Potential performance improvements. We'll get done whatever we can get done in 2 hours.

https://etherpad.wikimedia.org/p/instruments_observations

( Note: Lead image performance was addressed with this patch: https://phabricator.wikimedia.org/T91328 )

Event Timeline

dr0ptp4kt assigned this task to Mhurd.
dr0ptp4kt raised the priority of this task from to Needs Triage.
dr0ptp4kt updated the task description. (Show Details)
dr0ptp4kt moved this task to Needs Estimation on the Wikipedia-iOS-App-Backlog board.
dr0ptp4kt subscribed.
dr0ptp4kt renamed this task from [SPIKE ? hrs] Facial detection speed-ups to [SPIKE ? hrs] Performance: Facial detection speed-ups.Feb 27 2015, 6:11 PM
dr0ptp4kt triaged this task as High priority.
dr0ptp4kt set Security to None.
dr0ptp4kt updated the task description. (Show Details)
KLans_WMF renamed this task from [SPIKE ? hrs] Performance: Facial detection speed-ups to [SPIKE 2 hrs] Performance: Facial detection speed-ups.Feb 27 2015, 7:09 PM
Mhurd removed Mhurd as the assignee of this task.Mar 3 2015, 7:28 PM

If you just do this in the background, then the image "jumps" when the face is detected.

To make this work without jumping, we would have to delay drawing the image. This is probably worse.

IMO I wouldn't delay the image drawing unless it was a last resort. First I would evaluate the other performance enhancements and see if they are enough.