<feed xmlns='http://www.w3.org/2005/Atom'>
<title>dcpomatic, branch video-range-bug</title>
<subtitle>DCP-o-matic DCP tools</subtitle>
<id>https://git.carlh.net/cgit/dcpomatic/atom?h=video-range-bug</id>
<link rel='self' href='https://git.carlh.net/cgit/dcpomatic/atom?h=video-range-bug'/>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/'/>
<updated>2022-08-11T21:48:40Z</updated>
<entry>
<title>Fix video range in preview and DCP when source is VIDEO rather than FULL.</title>
<updated>2022-08-11T21:48:40Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2022-08-11T21:42:01Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=2e607b7968a0af94d7e38fb9d7d5459e33beed85'/>
<id>urn:sha1:2e607b7968a0af94d7e38fb9d7d5459e33beed85</id>
<content type='text'>
It looks like the handling of "video" range (16-235) sources (when
creating "full" range (0-255) outputs) has been broken for some considerable
time.  Full range outputs are used for both preview and DCP creation.

We assumed that after we called sws_setColorspaceDetails FFmpeg would take
care of the conversion from video to full, but its source suggests that
it refuses to convert to full-range RGB (only full-range YUV or
greyscale).  Previously there was a seemingly correct comment (that
FFmpeg would do nothing if source _or_ destination were RGB) but the
check on when to use video_range_to_full_range() was only activated
for RGB sources, not (also) RGB outputs.

It's hard to believe that this has been around unnoticed for so long.

https://dcpomatic.com/forum/viewtopic.php?t=1928&amp;start=10 has a
discussion which pointed out the problem.
</content>
</entry>
<entry>
<title>Cleanup: don't repeat check inside resume() before calling it.</title>
<updated>2022-08-11T20:40:59Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2022-08-06T20:58:36Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=da75178e689a48dc689b70e0880a45f0cf8c07a8'/>
<id>urn:sha1:da75178e689a48dc689b70e0880a45f0cf8c07a8</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Allow cancellation of un-started jobs (#2777).</title>
<updated>2022-08-11T20:40:59Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2022-08-06T20:57:58Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=1fe717e9125e896c92f281922e87f8158b15c7af'/>
<id>urn:sha1:1fe717e9125e896c92f281922e87f8158b15c7af</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Cleanup: set job state to cancelled in a different place.</title>
<updated>2022-08-11T20:40:59Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2022-08-06T20:36:21Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=1d3085c403f265fb203654d461c0b77974205f2f'/>
<id>urn:sha1:1d3085c403f265fb203654d461c0b77974205f2f</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Fix d881836379172f4072ed81ea074f46b3a363a681 which removed clearing</title>
<updated>2022-08-11T14:39:03Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2022-08-11T14:39:03Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=f2b10879dcafccdb28f88d53dc1fc22094b96c1d'/>
<id>urn:sha1:f2b10879dcafccdb28f88d53dc1fc22094b96c1d</id>
<content type='text'>
of the left-hand-side of images.
</content>
</entry>
<entry>
<title>Lower some error thresholds now that check_image() actually works in these cases.</title>
<updated>2022-08-11T14:38:38Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2022-08-11T14:38:38Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=d4d9f1b9213cf6e95e840009488d84ad4a24b21b'/>
<id>urn:sha1:d4d9f1b9213cf6e95e840009488d84ad4a24b21b</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Fix broken writing of 16-bit images (the pixels must be written MSB-first).</title>
<updated>2022-08-11T14:38:24Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2022-08-11T14:38:24Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=8e4f083438e2a6234da96f63fcceadafd5a8bd03'/>
<id>urn:sha1:8e4f083438e2a6234da96f63fcceadafd5a8bd03</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Cleanup: remove duplicated code.</title>
<updated>2022-08-11T14:37:57Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2022-08-11T14:37:57Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=ddeecdac0600a3fe72660fb1e14b652c59f09d33'/>
<id>urn:sha1:ddeecdac0600a3fe72660fb1e14b652c59f09d33</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Bump libsub for fix to SSA colour tag parsing.</title>
<updated>2022-08-10T19:09:01Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2022-08-10T19:09:01Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=7abb914a871af91422a542e405d37c30bfa199f7'/>
<id>urn:sha1:7abb914a871af91422a542e405d37c30bfa199f7</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Fix lines down the right-hand-side of exports in some cases.</title>
<updated>2022-08-08T22:09:30Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2022-07-26T13:22:54Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=d881836379172f4072ed81ea074f46b3a363a681'/>
<id>urn:sha1:d881836379172f4072ed81ea074f46b3a363a681</id>
<content type='text'>
</content>
</entry>
</feed>
