View Bug Details

IDProjectCategoryView StatusLast Update
0001085DCP-o-maticBugspublic2018-10-17 20:16
ReporterCantar4 Assigned Tocarl  
PrioritynormalSeverityblockReproducibilityalways
Status closedResolutionfixed 
PlatformMacOSOS X OS Version10.11
Product Version2.11.0 
Target Version2.14.0 
Summary0001085: 2.11.10 preview mode: clicking the scrub button
Description

seems to disable the subtitle file content
sends current image TC backward.
Subtitles show again after 'playing' for a while

TagsNo tags attached.
Branch
Estimated weeks required
Estimated work requiredUnknown

Relationships

related to 0001204 acknowledged Try another subtitle approach where able decoders can bypass the pass() API and give subtitles accurately 

Activities

Cantar4

2017-06-29 16:57

reporter   ~0001729

there is a missing subtitle on "00:21.12", the last image of the file.
It can help you to know that
on the black frames following this image the count becomes continuous,
no more void subtitle on frames mm:ss.01.

carl

2017-07-10 10:09

administrator   ~0001733

Could you send me your source file?

Cantar4

2017-07-10 16:17

reporter   ~0001734

I am sending you 55MB by Wetransfer

  • Sarah-iPhone30-to-Handbrake24fps.mp4
  • All frames from 00m00s00i to 00m22s02i.srt

Cantar4

2017-07-13 16:11

reporter   ~0001738

if at 00:03.06 (or any other TC) you click on the scrollbar
the TC and the sub is reset to 00:00.00, but the scroll button stays in place (see pict.)

carl

2017-07-26 11:07

administrator   ~0001747

Non-moving scroll button should be fixed by f0a9735731740bf9da24847294d9781f1bc3a331

Cantar4

2017-08-15 12:51

reporter   ~0001765

Dynamic screen capture of 2.11.16 in PREVIEW : impossible to seek and fine tune a sub position.
(macOS 10.12.5)

carl

2018-01-15 20:48

administrator   ~0002001

Is this a unique bug or is it expressed by others that are open?

Cantar4

2018-01-16 03:05

reporter   ~0002010

Carl, i am under time pressure right now,
I will answer your bug 'closing' qestions all in one
by making a summary with '42 next week;

First of all, do you think my srt frame indexing file is a valid one
in terms of ms in/out TCs ?
--jp

carl

2018-01-16 08:28

administrator   ~0002013

Hi JP, there is no rush so take your time. The subtitle file you sent me does have a fault: "00:00.00i start" has a in and out time of 0, and 00:00.01i has a start time of 0 and a end time of 1 (frame). So I think something is wrong with the creation of subtitles.

Cantar4

2018-01-16 08:56

reporter   ~0002014

something is unclear, for me the millisec duration is OK,
proof is they work for the next second and all others.

do i have to start at 01.041 ?

1
00:00:00,000 --> 00:00:00,041
00:00.00i start

2
00:00:00,041 --> 00:00:00,083
00:00.01i

3
00:00:00,083 --> 00:00:00,124
00:00.02i

4
00:00:00,124 --> 00:00:00,166
00:00.03i

Cantar4

2018-01-18 01:55

reporter   ~0002028

I attributed a TC lenght of 41 or 42 ms to each srt sub (1000/24= 41.66ms.
it works in Preview and in exported mp4 or Prores files.
but no way to make it work on the standalone Player ; so I tested 1ms TC gaps (or overlaps)
between subs, the result is NG.

These tests let me think that subtitle duration vs playback frame period is very tricky.
Could it be that a 24fps .srt file becomes not acceptable to the Standalone Player
because a minute frequency discrepency progressively makes the frontier be trespassed
and subtitles become impossible to sync ?

Cantar4

2018-02-06 19:12

reporter   ~0002078

Subtitlewise, 2.11.49 is quite close to 2.12.0 :)
INTEROP metasubs are accurate and continuous*
While in pause, clicking on the timeline no longer sends back the player to the 00:00:00 start frame
but it makes the sub unpleasantly disappear ; you must click --> or <-- to see it back.

SMPTE metasubs still show a +1 frame error and disappear right after the 00:01.23 mark.

  • there is no more void on hh:mm:ss.00 frames, provided there is a 2ms overlap
    between the subtitle TC and the frame count of these frames.
DoM-Player 11-49.jpg (57,620 bytes)   
DoM-Player 11-49.jpg (57,620 bytes)   

Cantar4

2018-02-08 01:09

reporter   ~0002084

2.11.50 test
SMPTE metasubs are now accurate and show to the end of the file...
but only on mm:ss.00 frames (on which the TC overlaps 1ms ahead of the frame border),
so tomorrow I will make another test with a DCP built with a modified .srt in which I will
apply this 1ms offset to all frames.

BurnBlu-SoftOrange-Smpte-50.jpg (28,744 bytes)   
BurnBlu-SoftOrange-Smpte-50.jpg (28,744 bytes)   

Cantar4

2018-02-09 13:20

reporter   ~0002098

  • countingwise, 2.11.51 meta subs are perfect in both smpte and interop flavors.
  • the volatiliry of the subs which needed a + or - 1 frame nudge to reappear after a jump
    on the timeline is now solved in smpte but not in interop.

Carsten

2018-02-09 14:27

manager   ~0002102

I understand it is complicated to cater for all aspects of subtitle rendering including cueing. Ideally, DCP-o-matic would need to carry a state of all titles (search backwards and recue for upcoming subs). I guess Carl has good understanding of what a player should do to enable this, but the question is, is it compatible with the way the player works currently. For me, it would be sufficient if the player renders subs correctly during a normal playout. I understand that may be not enough for quality checks. However, subtitle files/timing should probably be qc'd in the subtitle application?

  • Carsten

Cantar4

2018-02-09 14:38

reporter   ~0002104

2.11.52 (Main & Player) : same interop subtitle volatiliy : needs [->] or [<-] for sub to reappear
after a time jump on the scrollbar.
Btw, would it be possible to also use the keyboard real arrows for +/- 1 frame scrubing?

Carsten

2018-02-09 15:00

manager   ~0002107

I just filed a feature request for simple keyboard actions like play/pause, frame left/right.

I fear that constantly monitoring subtitle display status (including all display aspects like fade, etc.) could slow down the player considerably?

carl

2018-02-09 17:08

administrator   ~0002110

11a6decd652158b2288de17dc1aeafcd860fdfef should fix disappearing subs on seek.

Cantar4

2018-02-09 18:09

reporter   ~0002111

is this "d860fdfef should fix disappearing subs on seek."
also works for DpM preview?

wink wink, say no more

carl

2018-02-09 18:20

administrator   ~0002112

Should do, it's the same code.

Cantar4

2018-02-12 06:28

reporter   ~0002135

should do but doesn't, a great expectation is now down to earth ;-)

no worry, we will patiently wait for 2.13.x, your 2.12 standalone player is now
such a jewel to play with !

carl

2018-02-12 12:33

administrator   ~0002136

Are you looking at DCP content in the preview, or is it your video/srt combination?

Carsten

2018-02-12 15:30

manager   ~0002137

Last edited: 2018-02-12 16:20

It works for me in 2.11.55 three ways:

  • in the standalone player - softsubs show up even while dragging the slider (if they are set long enough, of course ;-)
  • in DCP-o-matic with the project loaded (e.g. with a subless DCP and an added srt file)
  • in DCP-o-matic when I load the subtitled DCP

They also appear/disappear properly when nudging around their entry point.
It is not 100%, they don't always reappear when cueing backwards over their exit point. But most of the time ;-) The philosophical question is, is an exit point backwards still an exit point, or an entry point ;-)

That's in 2.11.55/Sierra. And no, luckily, there seems to be no performance penalty as I had feared.

  • Carsten

carl

2018-02-12 20:53

administrator   ~0002138

If it's your Sarah-iPhone the problem is caused by I-frames being 12 seconds apart, so a seek to 11s ends up being to 0 and so there is no subtitle. I have an idea for how to fix this (0001204) but I think it's too disruptive to include in 2.12.0 at this late stage. I'll bump this to 2.13.x unless I see a nice way to do 0001204 in the mean time.

carl

2018-08-16 22:07

administrator   ~0002620

I'm fairly sure this stuff is resolved and/or moved to other bugs; do let me know if not.

Bug History

Date Modified Username Field Change
2017-06-09 12:35 Cantar4 New Bug
2017-06-29 16:57 Cantar4 File Added: Screen-Shots-2017-06-21-w.jpg
2017-06-29 16:57 Cantar4 Note Added: 0001729
2017-07-09 19:01 Cantar4 Sponsorship Added Cantar4: US$ 50
2017-07-09 19:01 Cantar4 Sponsorship Total 0 => 50
2017-07-10 10:09 carl Assigned To => carl
2017-07-10 10:09 carl Status new => feedback
2017-07-10 10:09 carl Note Added: 0001733
2017-07-10 16:17 Cantar4 Note Added: 0001734
2017-07-10 16:17 Cantar4 Status feedback => assigned
2017-07-10 16:20 Cantar4 Sponsorship Updated Cantar4: US$ 5
2017-07-10 16:20 Cantar4 Sponsorship Total 50 => 5
2017-07-10 16:21 Cantar4 Sponsorship Updated Cantar4: US$ 55
2017-07-10 16:21 Cantar4 Sponsorship Total 5 => 55
2017-07-13 16:11 Cantar4 File Added: Screen-Shot-2017-07-13-at-06.49.47.jpg
2017-07-13 16:11 Cantar4 Note Added: 0001738
2017-07-26 11:07 carl Note Added: 0001747
2017-08-15 12:51 Cantar4 File Added: CaptureDCP-2-11-16 in Preview.mp4
2017-08-15 12:51 Cantar4 Note Added: 0001765
2017-08-15 23:09 carl Target Version => 2.12.0
2018-01-15 20:48 carl Note Added: 0002001
2018-01-16 03:05 Cantar4 Note Added: 0002010
2018-01-16 08:28 carl Note Added: 0002013
2018-01-16 08:56 Cantar4 Note Added: 0002014
2018-01-18 01:55 Cantar4 Note Added: 0002028
2018-02-06 19:12 Cantar4 File Added: DoM-Player 11-49.jpg
2018-02-06 19:12 Cantar4 Note Added: 0002078
2018-02-08 01:09 Cantar4 File Added: BurnBlu-SoftOrange-Smpte-50.jpg
2018-02-08 01:09 Cantar4 Note Added: 0002084
2018-02-09 13:20 Cantar4 Note Added: 0002098
2018-02-09 14:27 Carsten Note Added: 0002102
2018-02-09 14:38 Cantar4 Note Added: 0002104
2018-02-09 15:00 Carsten Note Added: 0002107
2018-02-09 17:08 carl Note Added: 0002110
2018-02-09 18:09 Cantar4 Note Added: 0002111
2018-02-09 18:20 carl Note Added: 0002112
2018-02-12 06:28 Cantar4 Note Added: 0002135
2018-02-12 12:33 carl Note Added: 0002136
2018-02-12 15:30 Carsten Note Added: 0002137
2018-02-12 16:20 Carsten Note Edited: 0002137
2018-02-12 20:50 carl Relationship added related to 0001204
2018-02-12 20:50 carl Target Version 2.12.0 => 2.14.0
2018-02-12 20:53 carl Note Added: 0002138
2018-08-16 22:07 carl Status assigned => resolved
2018-08-16 22:07 carl Resolution open => fixed
2018-08-16 22:07 carl Note Added: 0002620
2018-10-17 20:16 carl Status resolved => closed