octave-patch-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-patch-tracker] [patch #9714] [octave forge] (image) new function


From: Carnë Draug
Subject: [Octave-patch-tracker] [patch #9714] [octave forge] (image) new function bwunpack
Date: Wed, 7 Nov 2018 08:39:52 -0500 (EST)
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0

Update of patch #9714 (project octave):

                  Status:                    None => Done                   
             Open/Closed:                    Open => Closed                 

    _______________________________________________________

Follow-up Comment #1:

I have pushed your patch and made some more changes.  They are all listed on
the commit message http://hg.code.sf.net/p/octave/image/rev/d2145f41596c

The main ones are:

1) don't check if m exists. Just check nargin and set it as default if
missing

2) add a check for M being larger than the number of bits on each column. 
This is Matlab incompatible but in my opinion a bug in Matlab. I got it tested
and seems that Matlab results are not reproducible. I guess that Matlab must
be unpacking whatever's in memory after the packed image.

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/patch/?9714>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]