Skip to content

Add AutoGrouper #178

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

Merged
merged 3 commits into from
May 6, 2019
Merged

Conversation

BzKevin
Copy link
Contributor

@BzKevin BzKevin commented Mar 4, 2019

Adds AutoGrouper script which allows the user to select multiple images (or an entire collection) and automatically group them based on timestamp. The separation between groups is user defined via a slider.

BzKevin added 2 commits March 4, 2019 12:20
Adds AutoGrouper script which allows the user to select multiple images (or an entire collection) and automatically group them based on timestamp. The separation between groups is user defined via a slider.
Added comments throughout
Change the gap comparison to be inclusive ( from '<' to '<=')
label = _('group gap [sec.]'),
tooltip = _('minimum gap, in seconds, between groups'),
soft_min = 1,
soft_max = 30,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this high enough? Should it maybe be a minute or two, or five? I tried it on a baseball game I shot and I ended up at 200 seconds. I tried 300, but that was a little too much. I also tried a 2 basketball games shot back to back. With a 30 second gap I got 45 groups. I think the right gap for that is probably 60 seconds, which would group between timeouts. Lastly I tried a senior picture session I did. The 30 second grouping was just about right and the grouping was pretty close to what I did manually when I processed them.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The hard max is 3600 so the user can enter up to a full 1hr gap. I don't really know what would make a good soft max.

My only thought was to avoid a frustration I have with some other sliders where the adjustments I am trying to make are difficult due to being stuck on the far end of the slider range. This happens to me frequently when using the right-click method for adjusting white balance, It's hard to make fine tune adjustments when lowering WB because I'm already so close to the left-hand boundary.

That being said, I don't have a strong preference here, so if you all feel a higher soft max would be more usable by the majority of people then let's make it whatever you think is best.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure either. I toyed with the idea of 3 sliders, hours, minutes, and seconds, but I'm not sure that's any better. How about we make it 60 seconds, and then see if the users provide any feedback. Or, we can leave it as is and see if the user's have feedback. Up to you. Let me know.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

60 sounds good, a nice even 1 minute. I'll do that and take care of these other notes you've made when I get some time to tinker with these again. Glad you got some use out of it though.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made it 60, I also increased the hard max all the way up to a full day. I saw no drawbacks to having a very large hard max.

1) mod changed to MOD
2) autogroup changed to autogrouper
3) prints "please select at least 2 images" if user attempts to run with fewer than 2 images selected
4) change visible module name to "auto group" (lower case)
5) changed soft max on slider to 60s
         - I also increase the hard max up to a full day, I saw no reason not to
Copy link
Member

@wpferguson wpferguson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@wpferguson wpferguson merged commit 6ab029e into darktable-org:master May 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants