wolfgangw / backports

Deep inspection of digital cinema packages
27 stars 12 forks source link

Remove hint "DCSubtitle x: ReelNumber 'y' does not match its CPL reel number 'z'"? #100

Closed matmat closed 9 months ago

matmat commented 10 months ago

https://github.com/wolfgangw/backports/blame/7360845592e3ba9734c90225cfa48595fb6289cf/dcp_inspect#L3038-L3041

To be honest I find this hint a bit noisy. It has never really helped me finding anything bad.

More often than not this is a false positive when an company logo or similar is inserted at the start of the composition. E.g. like in this case:

Reel 1:
   680  00:00:28+08 @ 24.0  Entry 00:00:10+00  fad64417  MainPicture    (MXF Interop, 00:00:48+08, plaintext, 1998x1080, avg 17.35 Mb/s, max 177.61 Mb/s, pictures)
   680  00:00:28+08 @ 24.0  Entry 00:00:00+00  6c6eda56  MainSound      (MXF Interop, 00:00:28+08, plaintext, 6ch 48kHz 24bps, RMS -inf dBFS |.|.|.|.|.|.|, Peak -inf dBFS |.|.|.|.|.|.|, audio)
Reel 2:
 22117  00:15:21+13 @ 24.0  Entry 00:00:10+00  725e5e83  MainPicture    (MXF Interop, 00:15:41+13, plaintext, 1998x1080, avg 79.08 Mb/s, max 195.44 Mb/s, pictures)
 22117  00:15:21+13 @ 24.0  Entry 00:00:02+00  8719ce9c  MainSound      (MXF Interop, 00:15:25+13, plaintext, 6ch 48kHz 24bps, RMS -32.64 dBFS |▃|▃|▄|▃|▃|▃|, Peak -1.00 dBFS |▇|▆|█|▆|▅|▆|, audio)
 22117  00:15:21+13 @ 24.0  Entry 00:00:00+00  b6251989  MainSubtitle   (DCSubtitle, 150 subtitles, 00:01:09+12 'This is to [...]' - 00:14:44+16 'It's alright.')
Reel 3:
 26275  00:18:14+19 @ 24.0  Entry 00:00:10+00  26360a88  MainPicture    (MXF Interop, 00:18:34+19, plaintext, 1998x1080, avg 72.84 Mb/s, max 190.26 Mb/s, pictures)
 26275  00:18:14+19 @ 24.0  Entry 00:00:02+00  dbc77c8c  MainSound      (MXF Interop, 00:18:18+19, plaintext, 6ch 48kHz 24bps, RMS -31.27 dBFS |▃|▃|▄|▃|▃|▃|, Peak -0.99 dBFS |█|█|█|█|█|▇|, audio)
 26275  00:18:14+19 @ 24.0  Entry 00:00:00+00  b09e72da  MainSubtitle   (DCSubtitle, 151 subtitles, 00:00:09+17 'You ok?' - 00:18:10+16 'Is everything ok?')
Reel 4:
 25920  00:18:00+00 @ 24.0  Entry 00:00:10+00  fc0bb7a4  MainPicture    (MXF Interop, 00:18:20+00, plaintext, 1998x1080, avg 47.83 Mb/s, max 195.12 Mb/s, pictures)
 25920  00:18:00+00 @ 24.0  Entry 00:00:02+00  c137a280  MainSound      (MXF Interop, 00:18:04+00, plaintext, 6ch 48kHz 24bps, RMS -32.60 dBFS |▃|▃|▄|▃|▃|▃|, Peak -0.98 dBFS |▆|▆|█|▄|▅|▄|, audio)
 25920  00:18:00+00 @ 24.0  Entry 00:00:00+00  42445f4e  MainSubtitle   (DCSubtitle, 114 subtitles, 00:00:34+13 'That's the heart.' - 00:17:57+19 'I owe her [...]')
Reel 5:
 23264  00:16:09+08 @ 24.0  Entry 00:00:10+00  8913efdf  MainPicture    (MXF Interop, 00:16:29+08, plaintext, 1998x1080, avg 35.34 Mb/s, max 193.52 Mb/s, pictures)
 23264  00:16:09+08 @ 24.0  Entry 00:00:02+00  5974f7b9  MainSound      (MXF Interop, 00:16:13+08, plaintext, 6ch 48kHz 24bps, RMS -35.83 dBFS |▃|▃|▄|▃|▃|▃|, Peak -0.99 dBFS |▆|▆|█|▄|▄|▄|, audio)
 23264  00:16:09+08 @ 24.0  Entry 00:00:00+00  aeddea0a  MainSubtitle   (DCSubtitle, 140 subtitles, 00:00:23+09 'Eat.' - 00:16:08+12 'Please.')
Reel 6:
 22835  00:15:51+11 @ 24.0  Entry 00:00:10+00  265dd814  MainPicture    (MXF Interop, 00:16:01+11, plaintext, 1998x1080, avg 36.25 Mb/s, max 155.39 Mb/s, pictures)
 22835  00:15:51+11 @ 24.0  Entry 00:00:02+00  cc6bdc5d  MainSound      (MXF Interop, 00:15:53+11, plaintext, 6ch 48kHz 24bps, RMS -30.19 dBFS |▄|▄|▄|▃|▃|▃|, Peak -0.98 dBFS |▇|█|█|▅|▅|▅|, audio)
 22835  00:15:51+11 @ 24.0  Entry 00:00:00+00  2aa9f2fc  MainSubtitle   (DCSubtitle, 53 subtitles, 00:01:13+07 'Santiago, stop yelling.' - 00:15:45+12 'Subtitling TITRAFILM')
Total duration:
121091  01:24:05+11 @ 24.0

Maybe it is time to remove this hint altogether?

wolfgangw commented 9 months ago

I agree, with a heavy heart :)

Actually, after reading the spec again, I'm not comfortable with taking it out. In relation to Presentation List, where ReelNumber is - among other elements - required, this may break stuff with caption systems, I can't say. Leaving it in.

Deluxe never cared, SMPTE made ReelNumber informational/optional.

wolfgangw commented 9 months ago

And Netflix has The “ReelNumber” in a Timed Text XML must match the corresponding CPL’s reel/segment number. in their Digital Cinema Package (DCP): Specifications & Requirements

matmat commented 9 months ago

Ok! Fine with me :)