darktable icon indicating copy to clipboard operation
darktable copied to clipboard

Increase culling mode zoom limit to facilitate easier culling on modern systems

Open CritLoren opened this issue 10 months ago • 17 comments

Is your feature request related to a problem? Please describe. Coming from Capture One, where I've developed an easy way to cull photos by comparing all of them (up to 12 at a time), deselecting the sharpest image, and flagging the rest for deletion, i've struggled to replicate it in Darktable. I can compare images (albeit a bit slower to select) but I cannot pixel peep as easily, seeing as zooming is forcefully disabled over 4 images.

Describe the solution you'd like Increasing the limit for the culling mode zoom.

Alternatives Currently I'm importing my photos through Capture one, then switching back to my linux install, it works, but i'd rather I didn't have to. If there's something I've missed, let me know

Additional context This is how easy it is for me to compare for culling on capture one: https://github.com/user-attachments/assets/944c0bf6-9fa2-4a90-855f-386f18914550

CritLoren avatar Mar 12 '25 23:03 CritLoren

This issue has been marked as stale due to inactivity for the last 60 days. It will be automatically closed in 300 days if no update occurs. Please check if the master branch has fixed it and report again or close the issue.

github-actions[bot] avatar May 13 '25 00:05 github-actions[bot]

Not this bot here too...

CritLoren avatar May 13 '25 00:05 CritLoren

This issue has been marked as stale due to inactivity for the last 60 days. It will be automatically closed in 300 days if no update occurs. Please check if the master branch has fixed it and report again or close the issue.

github-actions[bot] avatar Jul 13 '25 00:07 github-actions[bot]

Crazy, cause the issue isn't fixed yet

CritLoren avatar Jul 13 '25 02:07 CritLoren

You're more than welcome to contribute a fix.

Crazy, cause the issue isn't fixed yet

You may not have noticed, but in the last 60 days we built a release and released it. Now we concentrating on fixing bugs in the release.

As far as marking the issue as stale, that flags the issue so that it's brought to the developers attention again. It actually works well in this case since the issue was submitted when people were trying to get the release finished.

As far as the issue itself...

  • I'm sure at this point no one knows how to implement it
  • I'm not sure it can be implemented without large changes to the UI
  • I'm sure it's not going to be fixed in 60 days.

wpferguson avatar Jul 13 '25 02:07 wpferguson

I'm not knowledgeable enough to implement a fix for this. The attitude in my comment was towards the use of a bot to auto close issues, not towards this issue not getting attention.

CritLoren avatar Jul 13 '25 02:07 CritLoren

the use of a bot to auto close

If we didn't use the bot then people would open an issue and if it didn't get picked up it would just sit there and most likely be forgotten about, other than by the submitter. The bot, at a minimum, resurfaces the issue at least twice. If someone comments, then the issue gets raised again in 60 days.

Our knowledge grows the more we do this and out interests may change, plus we also get new blood. So an issue that didn't interest us, or we didn't have an idea of how to solve, may get picked up the second, third, or fourth time around because we've grown and now we can figure it out.

wpferguson avatar Jul 13 '25 03:07 wpferguson

As far as the issue itself...

* I'm sure at this point no one knows how to implement it

* I'm not sure it can be implemented without large changes to the UI

* I'm sure it's not going to be fixed in 60 days.

I am curious though, isn't the cap something that can be simply edited? Is the UI not responsive to account for more images?

CritLoren avatar Aug 26 '25 19:08 CritLoren

Let me explain why the bot is useful.

You submit an issue. It may or may not get looked at.

After 60 days of no activity it gets flagged and a new email is generated saying there has been no activity for 60 days, but it gets put before the developers (and anybody that subscribes to the list) again to give them a second chance to look at it. After 300 more days of inactivity it gets flagged again and closed.

I've pulled several at the 300 days and closed, including one last week. I either didn't see them before, or I had no idea how to fix them. The one from last week I offered the OP a solution and he seems good with it. If the bot hadn't closed it, then it would be sitting and the OP wouldn't have a solution.

Right now there are 392 open issues. I don't think anyone is going to go thru all those issues looking for something to fix.

I am curious though, isn't the cap something that can be simply edited?

I don't think that it's that easy. @Solarer?

Is the UI not responsive to account for more images?

Depends on your hardware. I use a script with a timeout of 2 or 3 seconds. Another user told me they increased the timeout to 60 seconds because the script was on their old, slow laptop.

wpferguson avatar Aug 26 '25 20:08 wpferguson

I'm just curious why the cap on zooming was hard capped instead of allowed to go as high as possible based on user hardware.

CritLoren avatar Aug 26 '25 20:08 CritLoren

So that users didn't complain because their machine froze or darktable crashed when they zoomed too far (because they didn't have enough resources). If you don't think that's realistic, look at all the issues about darktable freezing with the blurs module when users set the radius too high, because it's not capped and probably should be.

EDIT: But then if you do cap it, you get issues wondering why it's capped. So, we can't win.

wpferguson avatar Aug 26 '25 20:08 wpferguson

Could add a warning when going over a certain threshold that it's unstable and require users to acknowledge that before increasing it further. Funny enough I was about to comment on one of the blur issues, also on it being capped :D

CritLoren avatar Aug 26 '25 20:08 CritLoren

If we had lots of hardware to test on, then we might be able to figure out "limits". But, we don't so we aim for what should work on most.

Could add a warning

We've tried that in the past. Then we removed the setting because no one thought it applied to them.

wpferguson avatar Aug 26 '25 21:08 wpferguson

I think I know which setting you're referring to, but a non descriptive setting placed away from where the user might encounter its effects is different UX-wise from a prompt when the user hits the cap, which could be the current value of 4. Similar for blur. When the user is presented with the choice in response to an action they're doing, they'll more easily put 2 and 2 together. Ofc ideally the problematic action is also highlighted after, because many people have trained themselves to press "Yes" and not read prompts, but I think a general "<!> Performance might be degraded. Select fewer images to zoom through" could work

CritLoren avatar Aug 26 '25 21:08 CritLoren

The problem is that we don't know where to put the warning. On you machine it will one place, on mine another. Everyone's machine is different with different workloads, different memory, different CPU, different GPU, etc. We could weigh darktable down with a bunch of benchmarking code that runs on startup to see what your machine is capable of and then try and come up with reasonable settings based on that, but I don't think users want to wait a couple of minutes while their system is stress tested each time they start darktable.. And then you have the users like me who don't zoom during culling and don't use the blur's module, so we don't really care. :smile:

wpferguson avatar Aug 26 '25 22:08 wpferguson

I would absolutely love to have more images to zoom, especially because with raw+jpg doubles only 2 images are supported.

Would it be possible to show the warning during the "loading spinner" while calculating the zoom?

So if you have a slow machine and 10 images selected for zooming, it will calculate 1 minute, and during this time there is the explanation popup displayed ("Please wait while calculating... Select fewer images for improved performance") with an option to cancel the operation.

If you have a good machine and 6 images selected, you won't even recognize the message because the loading spinner only shows for half a second

quovadit avatar Oct 14 '25 06:10 quovadit

This issue has been marked as stale due to inactivity for the last 60 days. It will be automatically closed in 300 days if no update occurs. Please check if the master branch has fixed it and report again or close the issue.

github-actions[bot] avatar Dec 14 '25 00:12 github-actions[bot]

Maybe the limit could 4 for by default and a setting for an arbitrary number be under a "risky settings" menu?

EDIT: I am dumb, you can do that already in $HOME/.config/darktable/darktablerc

rodrigo-ceccato avatar Dec 18 '25 13:12 rodrigo-ceccato