#92 Movie thumbnail support

open
5
4 days ago
2012-12-20
Eduardo Rocha
No

Adding thumbnail generation to movies. Can be disabled at compile or runtime.

Discussion

<< < 1 2 (Page 2 of 2)
  • Eduardo Rocha
    Eduardo Rocha
    2013-11-11

    @chud, I was unable to download the file. The link you've sent redirects to a page.

    Please make sure you are using the latest version of ffmpedthumnailer and avcodec.

    Minidlna already uses IN_CLOSE_WRITE event to call the thumbnail generation. If you are copying more than one file, the generation of the first will ovelap the second copy operation.

    I'll upload a new patch for the latest commit in git fixing the bug (4).

     
    • TeHashX
      TeHashX
      2013-11-20

      Can you please provide the patch for v1.1.1?
      Thanks

       
      • Eduardo Rocha
        Eduardo Rocha
        2013-11-26

        here it goes the patch for the git head that should work on v1.1.1.

         
        • HPP8140
          HPP8140
          2014-03-07

          Hi, I'm getting some errors patching the following files with '
          minidlna_thumbnail_v1.1.1.patch':

          options.c
          options.h

          If someone would be so kind to send to me.

          Thanks.

           
  • Rafal Szczesny
    Rafal Szczesny
    2013-11-10

    Hi,
    Where is the patch for 1.1.0?

     
  • chud
    chud
    2013-11-12

    @Eduardo
    just tried - all links work?

    I was building with FFmpeg 2.0.2 "Nameless", libavcodec 55. 18.102, and latest ffmpegthumbnailer, (ffmpeg 2.1 doesn't work with minidlna), (I have tried several older releases of everything too.) To be honest I have given up now . [I develop a 3rd party router firmware and was hoping to incorporate your patch]

    I ended up using this torrent as my test 'Solaris [1972] 1080p Rus + Eng subs'

    It is a BR rip with ~14 mkv's in the Extra's folder. The default thumb size of 160 would always crash it processing 'solaris_00317.mkv', setting thumb to 200 would work fine on all, setting it to something else (cant remember what, maybe 120) would make it fail on a different mkv.

    Basically video_thumbnailer_generate_thumbnail_to_file() kills the minidlna thread dead if it tries something it dislikes. (i.e. presume fault with ffmpegthumbnailer/avcodec, I guess there isn't much you could do)

    Anyway cheers for your hard work!

     
  • Eduardo Rocha
    Eduardo Rocha
    2013-11-13

    @chud,

    I've tested with the solaris_00317.mkv with 160 and it worked fine here. In fact, I'm using this patch for more than 10 months now and I've never got a SEGFAULT. I'm using it on a raspberrypi with debian.

    Anyhow, I'll work on removing the ffmpegthumbnailer as soon as I've got some free time.

     
  • chud
    chud
    2013-11-14

    @eduardo, hmm thats interesting - any thoughts as to why the thumb size could have any effect? (I tried altering quality but made no difference). This is on MIPS incidently, ffmpeg had no special asm in it either. Anyway if you can remove need for ffmpegthumbnailer that hopefully will sort it so I wont worry about it now :)
    Also if you are interested it struggled on avi's (specifically Greys Anatomy S08 xvid is all I tried), it appeared to try and seek and fail then say was using the first frame (black). (or words to that effect it was a week or so ago and I cannot remember what it was exactly) It would be about an order of magnitude slower on these avi's as mkv's presubably as it tried to seek through whole file? Just out of interest is this normal or not?
    Thanks again, R

     
  • joao feijo
    joao feijo
    2013-11-28

    @eduardo rocha do you know the patch to correct folder structure. I am using it and it´s awesome for samsung tvs. I already use yours samsungES.patch and i will try this one tonight.
    Regards from Brasil

     
  • TeHashX
    TeHashX
    2014-02-05

    I've seen you added a lot of updates to git repository, can you please add this thumbnail patch too? Users have to enable manually thumbnails creation at compile time if needed.
    Thanks for your work

     
  • TeHashX
    TeHashX
    2014-03-07

    I'm trying to apply thumbnails patch to v1.1.2 but when I run make, I get this error:

    CC minidlna.o
    In file included from minidlna.c:85:
    options.h:67: error: expected ',' or '}' before 'FORCE_SORT_CRITERIA'
    minidlna.c: In function 'init':
    minidlna.c:711: error: 'FORCE_SORT_CRITERIA' undeclared (first use in this function)
    minidlna.c:711: error: (Each undeclared identifier is reported only once
    minidlna.c:711: error: for each function it appears in.)
    minidlna.c:714: error: 'MAX_CONNECTIONS' undeclared (first use in this function)
    make[2]: [minidlna.o] Error 1
    make[2]: Leaving directory /var/tmp/b' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory/var/tmp/b'
    make:
    [all] Error 2

     
  • Eduardo Rocha
    Eduardo Rocha
    2014-03-07

    here it goes the patch based on yesterday git tree. Should work with 1.1.2.

     
    • TeHashX
      TeHashX
      2014-03-08

      Thanks for the patch, I compiled and working fine on debian asuswrt mipsel routers http://forums.smallnetbuilder.com/showthread.php?t=14573, the only warning I get when I run make is:
      CC upnpsoap.o
      upnpsoap.c: In function 'parse_sort_criteria':
      upnpsoap.c:514: warning: 'saveptr' may be used uninitialized in this function
      CC upnpreplyparse.o

       
      Last edit: TeHashX 2014-03-08
  • kdeenkhoorn
    kdeenkhoorn
    2014-04-26

    Hi All,
    The patch for version 1.1.2 as downloaded tonight does not work for ubuntu 12.04.
    After applying the patch ./configure --enable-thumbnail warns about a non existing option --enable-thumbnail. Also the ffmpeg thumbnail libraries ar'nt checked although they are installed on the system.
    After starting this new build minidlna binary complains about the thumbnail options in the config file and no thumbs are created.
    Currently i'm using 1.0.25 with an earlier version of this patch without a problem. Hopefully it will be fixed because the extra sub directory option in 1.1.2 is my wish for a long time.

     
    Last edit: kdeenkhoorn 2014-04-26
  • kdeenkhoorn
    kdeenkhoorn
    2014-04-28

    Found it: i was missing the autoconf package and after installation that part was fixed.
    I dit ./configure --enable-thumbs --enable-static

    The static part failed during the make phase because of a missing -lgcc_s this was fixed editing:

    configure.ac:

    if test "$enableval" = "yes"; then
    STATIC_CFLAGS="-DSTATIC"
    AC_SUBST(STATIC_CFLAGS)
    STATIC_LDFLAGS="-Wl,-Bstatic"
    AC_SUBST(STATIC_LDFLAGS)
    AC_MSG_RESULT([yes])
    else

    to

    if test "$enableval" = "yes"; then
    STATIC_CFLAGS="-DSTATIC"
    AC_SUBST(STATIC_CFLAGS)
    STATIC_LDFLAGS="-Wl -Bstatic"
    AC_SUBST(STATIC_LDFLAGS)
    AC_MSG_RESULT([yes])
    else

    Now i have a working minidlna daemon 1.1.2 for my Samsung UE32C6800.

     
  • Раджа
    Раджа
    2014-06-15

    Can you update your patch for 1.1.3?

    Attached file is my version of the patch.

     
    Last edit: Раджа 2014-06-15
  • Eduardo Rocha
    Eduardo Rocha
    2014-09-04

    Hi folks, here it goes the patch for minidlna 1.1.4. This is the last time I'll post it here since it will probably never get into the main branch. Feel free to keep it updated. Cheers!

     
    Last edit: Eduardo Rocha 2014-09-04
    • TeHashX
      TeHashX
      2014-09-04

      Thanx for the patch, I wonder why is not included?

       
  • Eduardo Rocha
    Eduardo Rocha
    2014-09-04

    Patches regarding video thumbnail has been around for more than 2 years now. Over that time, ARM CPU got much faster. Yes, it will make the scan process take a while longer but full scan happens once in a while if you have inotify enabled.
    Anyhow, I've created a github repository to create features that will make my user experience better. Feel free to cherry pick them.

     
<< < 1 2 (Page 2 of 2)