[Mlt-devel] Image sequence question

2012-02-07 Thread j-b-m
Hello.

Trying to improve image sequence support in Kdenlive, I discovered that MLT 
cannot currently load a sequence with files like this:


image205.jpg
image206.jpg
image207.jpg
(...)

If we pass image%.3d.jpg as resource, MLT will look for:

image001.jpg, image002.jpg, etc until a max of 100 is reached. The QImage / 
Pixbuf code shows that a begin property should allow to set a start number 
for the sequence, for example:

melt image%.3d.jpg begin=205

However, it does not work since the filename parsing routine is called from 
producer_init, so the properties are not yet available to the producer and 
whatever value we assign to begin will not be used.

I have a patch to achieve what I want using that syntax:

melt image%.3d.jpg?205

The number after the '?' is the first frame of the sequence. However with that 
syntax the resource is not correctly identified as an image and by default is 
sent to the avformat producer. So we need to manually set the producer to 
qimage or pixbuf, which is not ideal...

Do you have any idea how this could be implemented in a better way? Or should 
I leave it like this and send the patch?

Thanks  regards
jb










--
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d
___
Mlt-devel mailing list
Mlt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mlt-devel


Re: [Mlt-devel] Image sequence question

2012-02-07 Thread Dan Dennedy
On Tue, Feb 7, 2012 at 10:08 AM, j-b-m j-...@users.sourceforge.net wrote:
 Hello.

 Trying to improve image sequence support in Kdenlive, I discovered that MLT
 cannot currently load a sequence with files like this:


 image205.jpg
 image206.jpg
 image207.jpg
 (...)

 If we pass image%.3d.jpg as resource, MLT will look for:

 image001.jpg, image002.jpg, etc until a max of 100 is reached. The QImage /
 Pixbuf code shows that a begin property should allow to set a start number
 for the sequence, for example:

 melt image%.3d.jpg begin=205

 However, it does not work since the filename parsing routine is called from
 producer_init, so the properties are not yet available to the producer and
 whatever value we assign to begin will not be used.

ah, I think at some time we needed to make sure all producers would
return null for invalid inputs, and that caused 'begin' to no longer
work

 I have a patch to achieve what I want using that syntax:

 melt image%.3d.jpg?205

 The number after the '?' is the first frame of the sequence. However with that
 syntax the resource is not correctly identified as an image and by default is
 sent to the avformat producer. So we need to manually set the producer to
 qimage or pixbuf, which is not ideal...

 Do you have any idea how this could be implemented in a better way? Or should
 I leave it like this and send the patch?

We could simply increase the hard-coded 100 to something much higher
like INT_MAX. If you can do that and see how long it takes to fail on
an invalid input, that would be great. Of course, it will actually
vary depending on the machine, but just to get an idea...

-- 
+-DRD-+

--
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d
___
Mlt-devel mailing list
Mlt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mlt-devel


Re: [Mlt-devel] [PATCH] fix thread exit on error and thread termination

2012-02-07 Thread Dan Dennedy
 2012/1/16 Maksym Veremeyenko ve...@m1stereo.tv:
 wrong test code where mlt_consumer_stop was called immediately after
 mlt_consumer_start caused that sideeffect.

 to resolve such condition i attached a patch that properly determinate
 consumer's thread finish and avoid writing /trailer/ if no /header was
 written..

My testing with sdl and decklink consumers showed that they were not
vulnerable to this, but indeed avformat was. I resolved this a little
differently, and committed it.

-- 
+-DRD-+

--
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d
___
Mlt-devel mailing list
Mlt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mlt-devel


[Mlt-devel] Git: fix race condition stopping avformat consumer

2012-02-07 Thread Dan Dennedy
 src/modules/avformat/consumer_avformat.c |   19 ++-
 1 files changed, 10 insertions(+), 9 deletions(-)

New commits:
commit de3106c7ea55b5fb52c8322536c2e94dbbad2cfa
Author: Dan Dennedy d...@dennedy.org
Date:   Tue Feb 7 20:15:31 2012 -0800

fix race condition stopping avformat consumer

Reported-by: Maksym Veremeyenko


--
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d
___
Mlt-devel mailing list
Mlt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mlt-devel


Re: [Mlt-devel] [PATCH] use sse2 instruction for line compositing

2012-02-07 Thread Dan Dennedy
2012/2/6 Maksym Veremeyenko ve...@m1stereo.tv:
 04.02.12 22:25, Dan Dennedy написав(ла):

 2012/2/3 Maksym Veremeyenkove...@m1stereo.tv:

 02.02.12 18:57, Maksym Veremeyenko написав(ла):

 Hi,

 attached patch perform line compositing for SSE2+ARCH_X86_64 build. It
 works for a case where luma is not defined...



 updated patch attached


 If I am not mistaken, this change reduces precision to 8 pixels. The
 existing transition is already limited to a 2 pixel precision, which I
 am not happy about. I do not want to further reduce the precision,
 give different results depending on CPU, and effectively introduce a
 regression, as far as the user is concerned. Maybe we should limit it
 to only
 apply when width is a multiple of 8. Then, it would still be used for
 fullscreen composite on most profiles' resolution.


 not exactly...

 sse2 code process 8-pixels-groups, tail with 1...7 pixels processed by
 native code - that why i did not create a standalone function but putted
 code into existing composite_yuv code...

 i am still testing it... may be you can take a look if all processed fine.

I started testing with the demos in the demo/ directory. It mostly
works; however, mlt_my_name_is crashes on me (and one time
mlt_squeeze_box):

(gdb) bt
#0  0x7fffdecced14 in composite_line_yuv (
dest=0x25bae60  ~\037~
~\037~!~-\177\064~2\177\065}5\200\064}3\200\062}3\201\064}6\201\063|0\202-|+\202+|+\202,|-\202/{0\203\060{1\203\061{1\203\061{1\203\061|2\202\062|1\202/{/\202\060{1\202-{,\202-{1\202\062{2\202\065{:\202\062{3\203\063{3\203\063z6\203;z@\203=z@\203BzD\203JzR\204WzW\204^ya\202fyj\202myn\202nyn\202wyv\202yy~\202\177y|\202|y\177\202{y{\202|y~\202\177y}\201|y}\201{y{\201{yy\201xyw\201wyx\201w{w\201t{p\201...,
src=0x2c621f0
\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200\020\200...,
width=0, alpha_b=0x2c16e00 ,
alpha_a=0x1ee9a30
\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377\377...,
weight=65535, luma=0x0, soft=0,
step=65535) at composite_line_yuv_sse2_simple.c:6


-- 
+-DRD-+

--
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d
___
Mlt-devel mailing list
Mlt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mlt-devel