{"action":"created","issue":{"url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/3753","repository_url":"https://api.github.com/repos/Beep6581/RawTherapee","labels_url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/3753/labels{/name}","comments_url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/3753/comments","events_url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/3753/events","html_url":"https://github.com/Beep6581/RawTherapee/pull/3753","id":213548962,"number":3753,"title":"Feedback request: a new processing pipeline for \"Fast Export\" mode","user":{"login":"agriggio","id":11982086,"avatar_url":"https://avatars2.githubusercontent.com/u/11982086?v=3","gravatar_id":"","url":"https://api.github.com/users/agriggio","html_url":"https://github.com/agriggio","followers_url":"https://api.github.com/users/agriggio/followers","following_url":"https://api.github.com/users/agriggio/following{/other_user}","gists_url":"https://api.github.com/users/agriggio/gists{/gist_id}","starred_url":"https://api.github.com/users/agriggio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agriggio/subscriptions","organizations_url":"https://api.github.com/users/agriggio/orgs","repos_url":"https://api.github.com/users/agriggio/repos","events_url":"https://api.github.com/users/agriggio/events{/privacy}","received_events_url":"https://api.github.com/users/agriggio/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":21,"created_at":"2017-03-11T18:29:23Z","updated_at":"2017-03-15T15:41:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Beep6581/RawTherapee/pulls/3753","html_url":"https://github.com/Beep6581/RawTherapee/pull/3753","diff_url":"https://github.com/Beep6581/RawTherapee/pull/3753.diff","patch_url":"https://github.com/Beep6581/RawTherapee/pull/3753.patch"},"body":"As the title says, this is not (yet) a real pull request, but more a request\r\nfor feedback.\r\n\r\nIn the last few days I've been playing with a new, maybe crazy, idea about the\r\n\"Fast Export\" mode of RT.\r\n\r\nA bit of context first. I've always liked the idea of a fast mode that trades\r\nspeed for quality, but I've never been very happy about the current fast\r\nexport mode. In my opinion, it gives up too much in features in order to\r\nachieve a (reasonable) speed advantage compared to the normal mode.\r\n\r\nSo, inspired by the `--hq` flag of DarkTable (its description, I haven't\r\nlooked at the code), I started playing with the idea of a custom processing\r\npipeline to be used for the Fast Export mode. The idea is that of resizing the\r\nimage as early as possible, and run the rest of the processing steps on the\r\nresized image. So, the more you scale down the output, the more you gain. This\r\nof course means that you also lose quality, but that is part of the \"contract\"\r\nof Fast Export: if you use it, you are ready to sacrifice some quality for\r\nspeed.\r\n\r\nAfter some tweaking, I've reached a point in which I think the feature is\r\nusable. Therefore, I'd like to see if there is somebody willing to give\r\nhis/her toughts about it (and maybe try it out). I've added a new check box in\r\nthe Fast Export tab (see screenshot), that enables the new pipeline. If you\r\nleave it unchecked, you get the old behaviour of Fast Export. If you enable\r\nit, the idea is that you can then turn of all the \"Bypass\" flags and still get\r\na reasonable performance boost (depending on how much you resize, of course)\r\nwhile at the same time obtaining an output that is much closer to the \"normal\"\r\none. For instance, on my machine, resizing to 1920x1200 (from a 6000x4000\r\noriginal), the new Fast Export is about 5x faster than the normal\r\npipeline. The quality is not too bad, especially if the ISO is not too high.\r\n\r\nOne final note. This required some refactoring of the processing pipeline code\r\nin `simpleprocess.cc`. I *think* (or at least hope :-) I did not break\r\nanything, but I'm not 100% sure...\r\n\r\n![fast-export-pipeline](https://cloud.githubusercontent.com/assets/11982086/23825787/906296ec-0690-11e7-9b5b-2cc46cd758db.png)\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/comments/286782740","html_url":"https://github.com/Beep6581/RawTherapee/pull/3753#issuecomment-286782740","issue_url":"https://api.github.com/repos/Beep6581/RawTherapee/issues/3753","id":286782740,"user":{"login":"agriggio","id":11982086,"avatar_url":"https://avatars2.githubusercontent.com/u/11982086?v=3","gravatar_id":"","url":"https://api.github.com/users/agriggio","html_url":"https://github.com/agriggio","followers_url":"https://api.github.com/users/agriggio/followers","following_url":"https://api.github.com/users/agriggio/following{/other_user}","gists_url":"https://api.github.com/users/agriggio/gists{/gist_id}","starred_url":"https://api.github.com/users/agriggio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agriggio/subscriptions","organizations_url":"https://api.github.com/users/agriggio/orgs","repos_url":"https://api.github.com/users/agriggio/repos","events_url":"https://api.github.com/users/agriggio/events{/privacy}","received_events_url":"https://api.github.com/users/agriggio/received_events","type":"User","site_admin":false},"created_at":"2017-03-15T15:41:48Z","updated_at":"2017-03-15T15:41:48Z","body":"Thanks for the feedback @iliasg! I don't know much about demosaicing, but I'll try to learn... :smile: "}}
{"actor":{"display_login":"agriggio"}}