{"action":"created","issue":{"url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/1627","repository_url":"https://api.github.com/repos/Beep6581/RawTherapee","labels_url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/1627/labels{/name}","comments_url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/1627/comments","events_url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/1627/events","html_url":"https://github.com/Beep6581/RawTherapee/issues/1627","id":100363654,"number":1627,"title":"CIECAM and HDR images","user":{"login":"Beep6581","id":5844619,"avatar_url":"https://avatars.githubusercontent.com/u/5844619?v=3","gravatar_id":"","url":"https://api.github.com/users/Beep6581","html_url":"https://github.com/Beep6581","followers_url":"https://api.github.com/users/Beep6581/followers","following_url":"https://api.github.com/users/Beep6581/following{/other_user}","gists_url":"https://api.github.com/users/Beep6581/gists{/gist_id}","starred_url":"https://api.github.com/users/Beep6581/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Beep6581/subscriptions","organizations_url":"https://api.github.com/users/Beep6581/orgs","repos_url":"https://api.github.com/users/Beep6581/repos","events_url":"https://api.github.com/users/Beep6581/events{/privacy}","received_events_url":"https://api.github.com/users/Beep6581/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Beep6581/RawTherapee/labels/Component-Quality","name":"Component-Quality","color":"ededed"}],"state":"closed","locked":false,"assignee":{"login":"Beep6581","id":5844619,"avatar_url":"https://avatars.githubusercontent.com/u/5844619?v=3","gravatar_id":"","url":"https://api.github.com/users/Beep6581","html_url":"https://github.com/Beep6581","followers_url":"https://api.github.com/users/Beep6581/followers","following_url":"https://api.github.com/users/Beep6581/following{/other_user}","gists_url":"https://api.github.com/users/Beep6581/gists{/gist_id}","starred_url":"https://api.github.com/users/Beep6581/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Beep6581/subscriptions","organizations_url":"https://api.github.com/users/Beep6581/orgs","repos_url":"https://api.github.com/users/Beep6581/repos","events_url":"https://api.github.com/users/Beep6581/events{/privacy}","received_events_url":"https://api.github.com/users/Beep6581/received_events","type":"User","site_admin":false},"milestone":null,"comments":25,"created_at":"2015-08-11T17:27:30Z","updated_at":"2016-05-22T14:02:11Z","closed_at":"2016-05-22T14:02:11Z","body":"Originally reported on Google Code with ID 1643\n```\nAnother patch for CIECAM\r\n\r\nCIECAM contains variables Q(brightness) and M (colorfullness) that are theoretically\nintended for HDR images (tone-mapping. ..)\r\n\r\nI plan to try two (three) possibilities:\r\n1) use \"Tone-mapping\" from Ben ... and see how it works with the variables Q and M\r\n2) if the result is not good, use code \"Luminance HDR\" (hard work !!)\r\n3) if the results are not good ... give up!!??\r\n\r\nI started adapting the code CIECAM order to possibly use the data for HDR images.\r\nI created a data category \"CieImage\" which contains pointers (with the same concept\nas \"LabImage\")\r\n**J (lightness)\r\n**C (chroma)\r\n**Q (brightness) ==>  for HDR\r\n**M (colorfullness) ==> for HDR\r\n**h (hue)\r\nI did not create pointer \"s\" to save memory.\r\n\r\nFinally, the creation of this variable will result, according to the image, to use\nadditional memory from 200M° to 500M°, which is not a problem in 64bit, but will cause\ntrouble in 32bit.\r\n\r\nI have not changed the code for \"tone-mapping\" (Ben) ...\r\nI just did a little cleaning and adapted the appeal process by replacing the Lab values\nby CIECAM values, plus a few other modifications made necessary by the use of CIECAM.\r\nAnd of course integration CIECAM !!\r\n\r\nTo activate it, of course, check the \"tone-mapping\", and also in CIECAM: Algorithm\n\"Brightness + Colorfullness\" ==> \"Tone-mapping with Q brightness CIECAM\"\r\n\r\nTo my surprise, the results \"CIECAM + Ben's Tone-mapping\" are very very good, so I\nabandoned the steps 2) and 3)\r\nIs finally obtained images comparable (or superior) to that obtained with \"PhotomatixPro64\"\nwhich is the reference HDR images (or LuminanceHDR).\r\n\r\n\r\nI enclose a link to a test image (_ASC4209.NEF) and pp3 file.\r\nhttp://jacques.desmis.perso.neuf.fr/RT/_ASC4209.NEF\r\n\r\nTry disabling the checkbox \"Tone-mapping with Q brightness CIECAM\" and see the difference.\r\n\r\nI tried to reduce memory consumption by dividing the code in 2. There is reduction\nin memory requirements, but there appears a transition between areas (not good...therefore\nabandon)\r\n\r\nOf course, this does not solve any problems already present in \"Tone-mapping\", nor\nlimitations CIECAM.\r\n:)\n```\n\nReported by `jdesmis` on 2012-12-10 12:16:04\n\n<hr>\n * *Attachment: [_ASC4209.NEF.pp3](https://storage.googleapis.com/google-code-attachments/rawtherapee/issue-1643/comment-0/_ASC4209.NEF.pp3)*\n * *Attachment: [tonemapcie10dec.patch](https://storage.googleapis.com/google-code-attachments/rawtherapee/issue-1643/comment-0/tonemapcie10dec.patch)*\n * *Attachment: [cieimage.cc](https://storage.googleapis.com/google-code-attachments/rawtherapee/issue-1643/comment-0/cieimage.cc)*\n * *Attachment: [cieimage.h](https://storage.googleapis.com/google-code-attachments/rawtherapee/issue-1643/comment-0/cieimage.h)*"},"comment":{"url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/comments/220834089","html_url":"https://github.com/Beep6581/RawTherapee/issues/1627#issuecomment-220834089","issue_url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/1627","id":220834089,"user":{"login":"Beep6581","id":5844619,"avatar_url":"https://avatars.githubusercontent.com/u/5844619?v=3","gravatar_id":"","url":"https://api.github.com/users/Beep6581","html_url":"https://github.com/Beep6581","followers_url":"https://api.github.com/users/Beep6581/followers","following_url":"https://api.github.com/users/Beep6581/following{/other_user}","gists_url":"https://api.github.com/users/Beep6581/gists{/gist_id}","starred_url":"https://api.github.com/users/Beep6581/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Beep6581/subscriptions","organizations_url":"https://api.github.com/users/Beep6581/orgs","repos_url":"https://api.github.com/users/Beep6581/repos","events_url":"https://api.github.com/users/Beep6581/events{/privacy}","received_events_url":"https://api.github.com/users/Beep6581/received_events","type":"User","site_admin":false},"created_at":"2016-05-22T14:02:11Z","updated_at":"2016-05-22T14:02:11Z","body":"Seems to have been committed already."}}
{"actor":{"display_login":"Beep6581"}}