agreed.


On 9 October 2013 12:31, Juan Martin Runge <notifications@github.com> wrote:

> In utils.js:
>
> > @@ -67,6 +67,27 @@ var populateDB = function() {
> >      */
> >  }
> >
> > +exports.filmstrip_pool = new fp.Pool({size: 1}, function (media, done) {
> > +    var path = require('path');
> > +    var ext = path.extname(media.file);
> > +    var dest = path.join(conf.Dirs.screenshots, media._id + ext);
> > +    logger.info('starting filmstrip', media.file);
> > +
> > +    var f = new ffmpeg();
> > +    f.run (media.file, dest, {
> > +        custom: [ '-an', '-r', '1', '-vf', 'scale=200:ih*200/iw', '-vcodec', 'copy' ]
>
> 2013/10/9 José Luis Di Biase <notifications@github.com>
>  In utils.js: > @@ -67,6 +67,27 @@ var populateDB = function() { > */ > }
> > > +exports.filmstrip_pool = new fp.Pool({size: 1}, function (media, done)
> { > + var path = require('path'); > + var ext = path.extname(media.file); >
> + var dest = path.join(conf.Dirs.screenshots, media._id + ext); > +
> logger.info('starting filmstrip', media.file); > + > + var f = new
> ffmpeg(); > + f.run (media.file, dest, { > + custom: [ '-an', '-r', '1',
> '-vf', 'scale=200:ih*200/iw', '-vcodec', 'copy' ] @lvidarte<https://github.com/lvidarte><
> https://github.com/lvidarte> So, change arguments to match required
> resolution with -s
> Also please change the -vcodec option to 'libx264' as, as Niv said, if we
> are changing fps and size we have to reencode for sure.
> … <#1419def6eca09a7d_>
>  — Reply to this email directly or view it on GitHub<
> https://github.com/inaes-tic/mbc-caspa/pull/173/files#r6858553> .
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/inaes-tic/mbc-caspa/pull/173/files#r6859359>
> .
>



-- 
Niv Sardi