patchGNU Octave - Patches: patch #7988, [octave forge] (video) aviread -...


patch #7988: [octave forge] (video) aviread - import a series of frames at a time

Submitted by:  None
Submitted on:  Mon 01 Apr 2013 08:59:03 AM UTC  
Category:  Forge : other Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  Andreas Weber <andy1978> Originator Email:  -email is unavailable-
Open/Closed:  Open

Add a New Comment(Rich Markup)

You are not logged in

Please log in, so followups can be emailed to you.


Thu 07 Jul 2016 01:28:20 AM UTC, comment #5:

If this is part of a package rather than core, this is probably not urgent, unless you expect it to need changes to the core in order to compile and install on a range of platforms.

Lachlan Andrew <lachlan>
Wed 06 Jul 2016 01:16:01 PM UTC, comment #4:

Lachlan, I think this is true.
Perhaps I find time the next weeks to adapt the patch to the new codebase...

Andreas Weber <andy1978>
Project MemberIn charge of this item.
Wed 06 Jul 2016 01:07:51 PM UTC, comment #3:

Andy, I take it that Anonymous has given up on this patch...

Lachlan Andrew <lachlan>
Thu 24 Dec 2015 04:21:34 PM UTC, comment #2:

This patch does no longer apply to the current source tree. There are also some problems with the changed Octave API. I know it's discouraging if there's no recognition of your patch for more than two years.

Can you provide a patch witch can be applied to the current source? If not I'll try to rewrite and integrate it.

Thanks, Andy

Andreas Weber <andy1978>
Project MemberIn charge of this item.
Tue 02 Apr 2013 06:05:41 PM UTC, comment #1:

I just realised that the array storing the frame numbers should be an uint array rather than a double, so

- unsigned int framenr = (unsigned int)args(1).scalar_value();
+ uint32NDArray nf = args(1).array_value();

Sorry for missing that.

Mon 01 Apr 2013 08:59:03 AM UTC, original submission:

Patch for the aviread function from the Octave-forge video package.

The function was changed to be compatible to the matlab version of aviread. Now it can read a series of frames specified by an index vector instead of only a single frame at a time. The returned value is now a struct array with two fields "cdata" containing the images' data in a height x width x 3 uint8 array and a "colormap" field which is, so far, just empty because indexed colors are not implemented in the AVHandler yet. This also speeds up the function by a factor of 3.

The Demo has been changed to illustrate the new behaviour.



(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:

Attached Files
file #27735: added by None (4KiB - text/x-diff)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -email is unavailable- added by mtmiller (Updated the item)
  • -email is unavailable- added by lachlan (Posted a comment)
  • -email is unavailable- added by andy1978 (Posted a comment)
  • -email is unavailable- added by None (Submitted the item)

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.




    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-04-10 mtmiller CategoryCore : other => Forge : other
    2018-04-10 mtmiller CategoryNone => Core : other
        Summaryaviread - import a series of frames at a time => [octave forge] (video) aviread - import a series of frames at a time
    2015-12-24 andy1978 Assigned toNone => andy1978
    2013-04-01 None Attached File- => Added, #27735

    Back to the top

    Powered by Savane 3.4