Page MenuHomePhabricator

thumbtime/seek thumbnailing broken with floating point offsets
Open, LowPublic


It seems that the seek param of thumbor is no longer working as expected.
A url like:,_Germany_2009.ogv/300px-seek=2.5-Parus_major,_Germany_2009.ogv.jpg

should return an image from 2.5 seconds into the movie File:Parus_major,_Germany_2009.ogv but instead it shows the midway frame of the video.

This used to work...

Actually it seems,_Germany_2009.ogv/300px-seek=2-Parus_major,_Germany_2009.ogv.jpg
still works, so it's the floating point parsing that this fails on..$163

has int() which does indeed fail on floating numbers. Having subsecond thumbtimes is actually useful, as there can be 24 up to 60 frames in a second and this frame picking can definitely make a big difference.

Related Objects

Event Timeline

TheDJ renamed this task from thumbtime/seek thumbnailing broken to thumbtime/seek thumbnailing broken with floating point offsets.Jul 18 2019, 7:46 PM
TheDJ updated the task description. (Show Details)

Change 524330 had a related patch set uploaded (by TheDJ; owner: TheDJ):
[operations/software/thumbor-plugins@master] Add decimal seek offset for videos

Gilles added a project: Performance-Team.

Came back from vacation today. There is a pending thumbor-plugins deployment pending for T226707: cwebp chokes on YCCK JPGs already. Once that has gone through, I will take care of this fix getting merged and deployed.

Change 524330 abandoned by TheDJ:
Add decimal seek offset for videos

abandoned because i can't move this forward. whoever wants to fix it can take over the patch

Change 568646 had a related patch set uploaded (by Brion VIBBER; owner: Brion VIBBER):
[operations/software/thumbor-plugins@master] Add decimal seek offset for videos

Krinkle subscribed.

Moving old Blocked column with unclear status to Inbox.

Aklapper changed the task status from Stalled to Open.Sat, Apr 6, 7:33 AM

The previous comments don't explain who or what (task?) exactly this task is stalled on ("If a report is waiting for further input (e.g. from its reporter or a third party) and can currently not be acted on"). Hence resetting task status, as tasks should not be stalled (and then potentially forgotten) for years for unclear reasons.