<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
On 01/03/2015 10:26 AM, Chad Parry wrote:<br>
<blockquote cite="mid:54A83467.2080309@chad.parry.org" type="cite">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<div class="moz-text-html" lang="x-western">
<blockquote type="cite">
<pre style="white-space: pre-wrap; color: rgb(0, 0, 0); font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On Sat, 03 Jan 2015 03:49:14 -0000, Chad Parry <<a moz-do-not-send="true" href="http://lists.mythtv.org/mailman/listinfo/mythtv-users">spam at chad.parry.org</a>> wrote:
<blockquote type="cite"><pre style="white-space: pre-wrap; color: rgb(0, 0, 0); font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><i>I created a pull request more than 6 months ago:
</i><i><a moz-do-not-send="true" href="https://github.com/MythTV/mythtv/pull/75.">https://github.com/MythTV/mythtv/pull/75.</a> It drastically improves the
</i><i>start-up time for large slide shows. I also have another pull request
</i><i>for a separate feature that is blocked on this one. My code conforms to
</i><i>the standards and just needs someone to merge it. Can someone please
</i><i>accept it? Thanks!</i>
</pre></blockquote>
The MythGallery plugin is currently deprecated because it uses the old UI.
The plan is to replace it with the "New Image Gallery" in 0.28 that uses
the new UI as well as Storage Groups and can support a services API. I'm
working on this at the moment but haven't yet finalised a version for
review. Master currently contains a incomplete version that doesn't work
very well.
Of course, the new Gallery doesn't preclude the continued use of the old
plugin. However,
I notice that some of the old UI has been deprecated so it may not
function if/when that is removed.
A cursory look at your patch suggests that the old plugin took a long time
to generate thumbnails before starting the slideshow. This isn't an issue
with the new design as the thumbnails are pre-generated. I haven't seen
your 2nd pull.
What issues/features does that address ?</pre>
</blockquote>
Thanks for the info. I'd be happy to help merge these features
into the new UI if that would be useful.<br>
<br>
The reason I made the first patch is because it took a long time
to recursively traverse directories and find all the photos. I
noticed it because I have thousands of NFS-mounted photos. This
may still be an issue in the new UI, unless there is also fast
lookup in the database. The issue is exacerbated by my second
feature, because it reads EXIF data from all the photos during
the preparation stage.<br>
<br>
The second feature is a "seasonal slideshow," (<a
moz-do-not-send="true"
href="https://code.mythtv.org/trac/ticket/12335">https://code.mythtv.org/trac/ticket/12335</a>).
It was really fun to have enabled at my house over the holidays.
It's like the random slideshow, except with a much larger weight
assigned to photos taken at the same time of year. Near
Christmas you get more Christmas photos, and near birthdays you
get more birthday photos.<br>
</div>
</blockquote>
Ping! Roger, can you accept my pull request, (<a
href="https://github.com/MythTV/mythtv/pull/75">https://github.com/MythTV/mythtv/pull/75</a>)?
Or does anyone know who can?<br>
</body>
</html>