<br><br>On Tuesday, July 13, 2021, Phyllis Smith via Cin <<a href="mailto:cin@lists.cinelerra-gg.org">cin@lists.cinelerra-gg.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-size:small">Not so sure about that after running more tests. <br></div><div class="gmail_default" style="font-size:small">Using 601Pal, mediainfo reports BT.601. <br></div><div class="gmail_default" style="font-size:small">Using 601Ntsc, medinao reports BT.709 !!<br></div><div class="gmail_default" style="font-size:small">As always, have to make sure old projects still work.</div></div></blockquote><div><br></div><div><br></div><div>its called color hell for this exact reason) </div><div><br></div><div>may be mediainfo a bit wrong (at least ver. you used - shoukd be recent on Fedora, but small fixes like this can go in anytime...) </div><div><br></div><div>see confusing code at (should be recent enough from 2020)</div><div><br></div><div><a href="https://sourceforge.net/p/mediainfo/discussion/297610/thread/0b6d2fd38f/?limit=25">https://sourceforge.net/p/mediainfo/discussion/297610/thread/0b6d2fd38f/?limit=25</a></div><div><br></div><div>=====</div><div><br></div><div>ha.</div><div><br></div><div>//---------------------------------------------------------------------------</div><div>extern const char* Mpegv_colour_primaries(int8u colour_primaries)</div><div>{</div><div> switch (colour_primaries)</div><div> {</div><div> case 1 : return "BT.709";</div><div> case 4 : return "BT.470 System M";</div><div> case 5 : return "BT.601 PAL";</div><div> case 6 : return "BT.601 NTSC";</div><div> case 7 : return "SMPTE 240M"; //Same as BT.601 NTSC</div><div> case 8 : return "Generic film";</div><div> case 9 : return "BT.2020"; //Added in HEVC</div><div> case 10 : return "XYZ"; //Added in HEVC 2014</div><div> case 11 : return "DCI P3"; //Added in HEVC 2016</div><div> case 12 : return "Display P3"; //Added in HEVC 2016</div><div> case 22 : return "EBU Tech 3213"; //Added in HEVC 2016</div><div> default : return "";</div><div> }</div><div>}</div><div><br></div><div>//---------------------------------------------------------------------------</div><div>extern const char* Mpegv_transfer_characteristics(int8u transfer_characteristics)</div><div>{</div><div> switch (transfer_characteristics)</div><div> {</div><div> case 1 : return "BT.709"; //Same as BT.601</div><div> case 4 : return "BT.470 System M";</div><div> case 5 : return "BT.470 System B/G";</div><div> case 6 : return "BT.601";</div><div> case 7 : return "SMPTE 240M";</div><div> case 8 : return "Linear";</div><div> case 9 : return "Logarithmic (100:1)"; //Added in MPEG-4 Visual</div><div> case 10 : return "Logarithmic (316.22777:1)"; //Added in MPEG-4 Visual</div><div> case 11 : return "xvYCC"; //Added in AVC</div><div> case 12 : return "BT.1361"; //Added in AVC</div><div> case 13 : return "sRGB/sYCC"; //Added in HEVC</div><div> case 14 : return "BT.2020 (10-bit)"; //Same a BT.601 //Added in HEVC, 10/12-bit difference is in ISO 23001-8</div><div> case 15 : return "BT.2020 (12-bit)"; //Same a BT.601 //Added in HEVC, 10/12-bit difference is in ISO 23001-8</div><div> case 16 : return "PQ"; //Added in HEVC 2015</div><div> case 17 : return "SMPTE 428M"; //Added in HEVC 2015</div><div> case 18 : return "HLG"; //Added in HEVC 2016</div><div> default : return "";</div><div> }</div><div>}</div><div><br></div><div>//---------------------------------------------------------------------------</div><div>extern const char* Mpegv_matrix_coefficients(int8u matrix_coefficients)</div><div>{</div><div> switch (matrix_coefficients)</div><div> {</div><div> case 0 : return "Identity"; //Added in AVC</div><div> case 1 : return "BT.709";</div><div> case 4 : return "FCC 73.682";</div><div> case 5 : return "BT.470 System B/G";</div><div> case 6 : return "BT.601"; //Same as BT.470 System B/G</div><div> case 7 : return "SMPTE 240M";</div><div> case 8 : return "YCgCo"; //Added in AVC</div><div> case 9 : return "BT.2020 non-constant"; //Added in HEVC</div><div> case 10 : return "BT.2020 constant"; //Added in HEVC</div><div> case 11 : return "Y'D'zD'x"; //Added in HEVC 2016</div><div> case 12 : return "Chromaticity-derived non-constant"; //Added in HEVC 2016</div><div> case 13 : return "Chromaticity-derived constant"; //Added in HEVC 2016</div><div> case 14 : return "ICtCp"; //Added in HEVC 2016</div><div> default : return "";</div><div> }</div><div>}</div><div><br></div><div>=====</div><div><br></div><div>note _transfer characteristics_ #1 said to be the same for bt. 709 and #6 bt. 601! (in comment) </div><div><br></div><div>can you post full mediainfo output for input video, and three other videos: unmodded Cin, modded and set to BT601 NTSC, set to BT601 pal? </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 12, 2021 at 2:26 PM Andrew Randrianasulu via Cin <<a href="mailto:cin@lists.cinelerra-gg.org" target="_blank">cin@lists.cinelerra-gg.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">According to<div><br></div><div><a href="https://kdenlive.org/en/project/color-hell-ffmpeg-transcoding-and-preserving-bt-601/" target="_blank">https://kdenlive.org/en/<wbr>project/color-hell-ffmpeg-<wbr>transcoding-and-preserving-bt-<wbr>601/</a></div><div><br></div><div>bt 601 pal and bt 601 ntsc are slightly different, I tried to update CinGG's definitions so now she have two distinct 601 colorspaces, but I still can be wrong (esp on kr, kb coeffecients) </div><div><br></div><div>can anyone test patch 3 and see if it makes any positive difference? </div><div><br></div><div><br></div>
-- <br>
Cin mailing list<br>
<a href="mailto:Cin@lists.cinelerra-gg.org" target="_blank">Cin@lists.cinelerra-gg.org</a><br>
<a href="https://lists.cinelerra-gg.org/mailman/listinfo/cin" rel="noreferrer" target="_blank">https://lists.cinelerra-gg.<wbr>org/mailman/listinfo/cin</a><br>
</blockquote></div></div>
</blockquote>