Gallery Name Creating Large Templates

[sklum]

If you enroll a large number of files (e.g. using a wildcard within a folder), the name for the gallery will be every file you enrolled. This makes template size potentially massive. Right now, a simple workaround is to use the Retain transform to only keep certain metadata (not Gallery) but that shouldn't be necessary.

[jklontz]

If you enroll from a folder, the command should look like br -enroll /path/to/folder, which will recursively enroll all files in the folder, instead of br -enroll /path/to/folder/* which the shell will expand to a long file list as you've discovered :). If you need to filter your files, a regexp can be specified: br -enroll /path/to/folder(*.jpg) -- see the implementation in EmptyGallery as this syntax of the regexp I provided may be slightly wrong.

[sklum]

I'm not thrilled with this response - having every file name appended to the Gallery metadata field in this situation is incorrect.

[jklontz]

I'm open to alternatives, but no elegant ones immediately come to mind for me.

[sklum]

That's why I opened an issue?

[jklontz]

K, one suggestion would be to name the gallery after the first and last file in the list. I'm not sure how easy that is to implement though -- I haven't looked at that part of the code in a while though. Another alternative is to detect really long gallery names, and then store an MD5 hash of the name in these cases. I think I'm leaning in favor of the hashing approach...

[sklum]

Hashing seems reasonable. Anything above 64 characters?

[jklontz]

Works for me -- I don't think gallery name is used that often anyway.

[lbestrowden]

I need to enroll a folder of files and filter them so only jpeg files are enrolled. -enroll path/to/folder/*.jpg works from the shell, but I need to do this from within the YouTube plugin with a list of arguments to execute. Is there a regular expression that can also be used in this case (not from the shell)?

[jklontz]

@lbestrowden Yes, see my first comment. br -enroll '/path/to/folder(*.jpg)' will avoid shell interpretation and be handled within OpenBR. For details, see EmptyGallery in gallery.cpp, in this case EmptyGallery::regexp = *.jpg.

[lbestrowden]

The correct syntax ended up being br -enroll '/path/to/folder[regexp=jpg]'. EmptyGallery uses EmptyGallery::regexp to match the suffix of the filenames. Is this the desired functionality, as it's not able to match an arbitrary regular expression?

[jklontz]

Lacey, please feel free to submit a fix -- desired functionality is
dictated by actual encountered needs :)

On Thu, Aug 1, 2013 at 12:52 PM, lbestrowden notifications@github.comwrote:

The correct syntax ended up being br -enroll '/path/to/folder[regexp=jpg].
EmptyGallery uses EmptyGallery::regexp to match the suffix of the
filenames. Is this the desired functionality, as it's not able to match an
arbitrary regular expression?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/72#issuecomment-21951739
.

[lbestrowden]

Ok, will do at some point. For my current purposes, matching the suffix is sufficient... say that 5 times fast. :)

[sklum]

Made the change in fa4557b.

[jklontz]

Just saw this message, so this can be closed now?