<br><br>On Tuesday, July 13, 2021, Andrea paz <<a href="mailto:gamberucci.andrea@gmail.com">gamberucci.andrea@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Finally I managed to patch! On Phyllis' suggestion I entered the<br>
absolute path of the files to patch every time it didn't find them.<br>
Below is the test I did.<br>
<br>
I downloaded the app mentioned from the link and created a video on<br>
smartphone; as mentioned in the link the video produced is: YUV,<br>
NTSC,color range:tv; primaries=BT 601 PAL; Transfer=BT 601 and<br>
Matrix:BT 601 (a horror in this day and age where broadcast worldwide<br>
uses HD and rec 709). In CinGG without patching you have a slight<br>
change in brightness going from 601 to 709 and vice versa (correct is<br>
601!); you have a bigger change going from Mpeg to Jpeg (correct is<br>
mpeg!).<br>
Applying the patch and importing the same video as before you have<br>
some difference between BT601 PAL and BT601 NTSC (correct is BT601<br>
NTSC). Both are different from BT 709. Thanks Andrew, you did a great<br>
job for me!</blockquote><div><br></div><div>one concern is... while input video claims itself to use  bt 601 pal primaries - you get correct result wuth bt 601 ntsc? may be it was tagged wrong, or ffmpeg flipped it? </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
In CinGG, to make a transcode there is the excellent ColorSpace plugin<br>
that works just for these changes of color spaces.<br>
I once asked if it was possible to integrate it into the "Transcode"<br>
function so that it would work for all project sources and not just<br>
the edits to which the plugin applies. But I think it's a complicated<br>
job....<br>
</blockquote><div><br></div><div>code is already written, even with OpenGL accelerated preview... so may be it can be reused, or just tell swscale (because we mostly use ffmpeg here) to do some colorspace changes for us... </div>