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

 
 

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

Submitter:  None
Submitted:  Mon 01 Apr 2013 08:59:03 AM UTC
   
 
Category:  Forge : other Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  andy1978 Originator Email:  -email is unavailable-
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Tue 23 Nov 2021 07:29:22 AM UTC, comment #6: 

video 2.0.0 is a complete rewrite using OpenCV functions so this patch can no longer be applied. Closing report

Andreas Weber <andy1978>
Group Member
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>
Group Member
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>
Group Member
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.

Anonymous
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.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #27735:  aviread.cc.patch 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)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-11-23 andy1978 Open/ClosedOpen Closed
    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 aviread.cc.patch, #27735

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code