{"action":"created","issue":{"url":"https://api.github.com/repos/mpv-player/mpv/issues/1563","labels_url":"https://api.github.com/repos/mpv-player/mpv/issues/1563/labels{/name}","comments_url":"https://api.github.com/repos/mpv-player/mpv/issues/1563/comments","events_url":"https://api.github.com/repos/mpv-player/mpv/issues/1563/events","html_url":"https://github.com/mpv-player/mpv/issues/1563","id":56687531,"number":1563,"title":":srgb or :icc-profile set causes black crush and strange gamma","user":{"login":"Cyberbeing","id":917535,"avatar_url":"https://avatars.githubusercontent.com/u/917535?v=3","gravatar_id":"","url":"https://api.github.com/users/Cyberbeing","html_url":"https://github.com/Cyberbeing","followers_url":"https://api.github.com/users/Cyberbeing/followers","following_url":"https://api.github.com/users/Cyberbeing/following{/other_user}","gists_url":"https://api.github.com/users/Cyberbeing/gists{/gist_id}","starred_url":"https://api.github.com/users/Cyberbeing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Cyberbeing/subscriptions","organizations_url":"https://api.github.com/users/Cyberbeing/orgs","repos_url":"https://api.github.com/users/Cyberbeing/repos","events_url":"https://api.github.com/users/Cyberbeing/events{/privacy}","received_events_url":"https://api.github.com/users/Cyberbeing/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"haasn","id":1149047,"avatar_url":"https://avatars.githubusercontent.com/u/1149047?v=3","gravatar_id":"","url":"https://api.github.com/users/haasn","html_url":"https://github.com/haasn","followers_url":"https://api.github.com/users/haasn/followers","following_url":"https://api.github.com/users/haasn/following{/other_user}","gists_url":"https://api.github.com/users/haasn/gists{/gist_id}","starred_url":"https://api.github.com/users/haasn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/haasn/subscriptions","organizations_url":"https://api.github.com/users/haasn/orgs","repos_url":"https://api.github.com/users/haasn/repos","events_url":"https://api.github.com/users/haasn/events{/privacy}","received_events_url":"https://api.github.com/users/haasn/received_events","type":"User","site_admin":false},"milestone":null,"comments":9,"created_at":"2015-02-05T15:50:39Z","updated_at":"2015-02-05T20:18:49Z","closed_at":null,"body":"If this is for some reason intentional, it would be nice to have an option to enable gamut correction and linear light scaling, without forced gamma correction (i.e. input=output gamma).\r\n\r\n[Normal Black/White Clipping] (http://i.imgbox.com/NBps4iL6.png)\r\n[:sRGB Black/White Clipping] (http://i.imgbox.com/s1QzLFGd.png)\r\n\r\n[Normal Color Steps] (http://i.imgbox.com/vK9wXpvd.png)\r\n[:sRGB Color Steps] (http://i.imgbox.com/VB75gN83.png)\r\n\r\n[:icc-profile Black/White Clipping] (http://i.imgbox.com/gdbig4me.png)\r\n[madVR 3DLUT Black/White Clipping] (http://i.imgbox.com/nTx8JZhA.png)\r\n\r\n[:icc-profile Color Steps] (http://i.imgbox.com/IcbEemZR.png)\r\n[madVR 3DLUT Color Steps] (http://i.imgbox.com/vKi8B7fn.png)\r\n\r\nThe most noticeable issue with *:srgb* and *:icc-profile* on these AVS HD 709 test clips, is rather significant  near-black crush and less distinction between some of the steps. Trying to fix things with the *:gamma=* option just makes things worse."},"comment":{"url":"https://api.github.com/repos/mpv-player/mpv/issues/comments/73118843","html_url":"https://github.com/mpv-player/mpv/issues/1563#issuecomment-73118843","issue_url":"https://api.github.com/repos/mpv-player/mpv/issues/1563","id":73118843,"user":{"login":"Cyberbeing","id":917535,"avatar_url":"https://avatars.githubusercontent.com/u/917535?v=3","gravatar_id":"","url":"https://api.github.com/users/Cyberbeing","html_url":"https://github.com/Cyberbeing","followers_url":"https://api.github.com/users/Cyberbeing/followers","following_url":"https://api.github.com/users/Cyberbeing/following{/other_user}","gists_url":"https://api.github.com/users/Cyberbeing/gists{/gist_id}","starred_url":"https://api.github.com/users/Cyberbeing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Cyberbeing/subscriptions","organizations_url":"https://api.github.com/users/Cyberbeing/orgs","repos_url":"https://api.github.com/users/Cyberbeing/repos","events_url":"https://api.github.com/users/Cyberbeing/events{/privacy}","received_events_url":"https://api.github.com/users/Cyberbeing/received_events","type":"User","site_admin":false},"created_at":"2015-02-05T20:18:49Z","updated_at":"2015-02-05T20:18:49Z","body":">:srgb is functionally equivalent to :icc-profile with an sRGB profile. You could cross-compare to using :icc-profile=sRGB.icc, and ditto for madVR.\r\n\r\nI don't know why I would cross compare against madVR, when there is no standard behavior for creating a 3DLUT. If I set my monitor profile to sRGB and cross-compare against Photoshop, output looks identical to my *normal* mpv image. Not crushed like the *:srgb* MPV image.\r\n\r\n>I did the math, and it seems to be correct\r\n\r\nI very much doubt Photoshop's math is incorrect. So you must be doing something other than sRGB->Linear->sRGB. I assume you are mixing a BT.1886 function in there somewhere, or assuming the encoded gamma is different from the output gamma when converting to and from linear light. Which brings me back to why I opened this issue:\r\n\r\n> If this is for some reason intentional, it would be nice to have an option to enable gamut correction and linear light scaling, without forced gamma correction (i.e. input=output gamma).\r\n\r\nmadVR for example offers linear light scaling and simple gamut correction without a 3DLut, which does not affect gamma. I'd like to see something like that supported in MPV."}}