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 : otherPriority: 5 - Normal
Status: NonePrivacy: 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>
Project Member
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>
Project Member
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 File(s):

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 click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Please enter the title of George Orwell's famous dystopian book (it's a date):



    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 10 Apr 2018 06:36:41 AM UTCmtmillerCategoryCore : other=>Forge : other
    Tue 10 Apr 2018 06:36:21 AM UTCmtmillerCategoryNone=>Core : other
      Summaryaviread - import a series of frames at a time=>[octave forge] (video) aviread - import a series of frames at a time
    Thu 24 Dec 2015 04:22:55 PM UTCandy1978Assigned toNone=>andy1978
    Mon 01 Apr 2013 08:59:03 AM UTCNoneAttached File-=>Added, #27735

    Back to the top

    Powered by Savane 3.1-cleanup1