Skip to content

Selection of untagged images #97

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
Sep 25, 2017

Conversation

jbvoelker
Copy link

When tagging my collection, I was looking for a way to find untagged images in darktable. All I found were older threads and a feature request from five years ago.

So I wrote a lua script which checks the collection for untagged images and adds them to the selection. Images having no tags at all or only tags added by darktable (e.g. darktable|format|cr2) are considered untagged.

Maybe this is interesting for some other users, or did I miss an official way to show untagged images?

…automatic tags added by darktable are also considered untagged
]]

local dt = require "darktable"
require "official/yield"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this "require" is not needed in this script.

dt.configuration.check_version(...,{3,0,0},{4,0,0},{5,0,0})

-- Tell gettext where to find the .mo file translating messages for a particular domain
gettext.bindtextdomain("select_untagged",dt.configuration.config_dir.."/lua/locale/")
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you have any translations, that you could add to the pr?

Copy link
Author

@jbvoelker jbvoelker Sep 14, 2017

Choose a reason for hiding this comment

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

Not yet, but I will add german and french translations.

Copy link
Author

Choose a reason for hiding this comment

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

Should I include the .mo file, or is it left to the user of the script to build it?

Copy link
Contributor

Choose a reason for hiding this comment

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

As we don't have a build system for the mo file I would add them.

Copy link
Author

Choose a reason for hiding this comment

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

I have added them to my branch.

Copy link
Contributor

Choose a reason for hiding this comment

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

then please make a new pull request

@supertobi supertobi merged commit 4d76006 into darktable-org:master Sep 25, 2017
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