Initial draft for high quality processing button, fron Jens. #675
Conversation
|
I don't see any issues tagged in the darktable repo for this... |
There was a problem hiding this comment.
A few comments, mostly for future reference since I'll probably copy-edit once it's merged anyway. Key thing though is that the bottom panel screenshot also needs updating for the new icon. IIRC when I did it I increased the DPI (to 150 or something) and the font size (to 15), and increased the size of the side panels, so that the image displayed clearly in the documentation.
| Darktable pixelpipes process the modules in slightly different ways, the thumbnail pipe as an example is optimized for speed. | ||
|
|
||
| The darkroom pixelpipes for the main canvas and the second monitor try to keep resources within limits by downscaling and by | ||
| restricting the processed data to what is actually displayed - in darktable speak this is the "region of interest" (ROI). |
There was a problem hiding this comment.
remove these mid-sentence line breaks
|
|
||
| Click the  icon to activate high quality processing in the pixelpipe. | ||
|
|
||
| Darktable pixelpipes process the modules in slightly different ways, the thumbnail pipe as an example is optimized for speed. |
There was a problem hiding this comment.
I generally try to avoid starting sentences with the word "darktable" so that I can maintain the idiosyncratic lowercase form without breaking language. If you can't think of a way to reword the sentence it's fine like this.
| The darkroom pixelpipes for the main canvas and the second monitor try to keep resources within limits by downscaling and by | ||
| restricting the processed data to what is actually displayed - in darktable speak this is the "region of interest" (ROI). | ||
|
|
||
| Yet - for exports we have to process the whole image - likely without downscaling - resulting in much larger processing times |
There was a problem hiding this comment.
Use a double-hyphen -- in markdown for dashes, though I would replace the first one with a comma
|
This documentation is for darktable-org/darktable#15910 |
|
This pull request has not had any activity in the past 60 days and will be closed in 365 days if not updated. Please verify it has no conflicts with the master branch and rebase if needed. Please add a comment if you need help or give permission to other people to finish your work. |
Made a PR for @jenshannoschwalm from darktable-org/dtorg#252