-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Can the pipe be more optimized for more speed? #18133
Comments
Can you provide some logs that indicate this? |
Is it not behaving this way for you? |
I can not seem to reproduce. |
Here are files showing a raw (CR2) with hard deblur in Diffuse/Sharpen and a Tone Curve instance with a raster mask. Switching off the Tone Curve is quick, switching is on is slow as DT re-calculates lots/all of the pipe.
Also note re. efficiency, though a relatively minor point, the histogram seems to be re-calculated unnecessarily... bring up any window on top of darktable, e.g. terminal, minimise it, and note this causes 4 extra lines in the terminal window, all re-drawing the histogram! At least on my system. toneCurveTest-terminal-log.txt The raw (CR2) is too big to upload, you can get it here - |
Yes - your rhetoric question "Can the pipe be more optimized for more speed" could be be answered with "YES - short pause - but how to do that safely?"
The histo redrawing is a completely different story btw. |
I noticed that with the 5.0.0 appimage, DT is doing a lot of recalculation which seems potentially unnecessary.
When you change something late in the pipe, it is prone to recalculating the whole pipe, whereas you might hope it would realise it's only from the point of change that the pipe needs recalculating.
To demonstrate -
Take an image with some chunky processing so the effect is clearly visible, e.g. DorS hard deblur. (Or do -d perf)
Add something late in the pipe like Local Contrast or Tone Curve with a raster mask that is an instance of say Exposure.
Now toggle Local Contrast off and on.
Notice that the offs are quick but the ons are slow because of pipe recalc.
Yet neither the raster mask, nor anything else, has changed prior to Local Contrast.
If you're using Tone Curve to make some final tweaks, it is quite frustrating!
Ubuntu 22.04, no GPU card
The text was updated successfully, but these errors were encountered: