|
From: | Carnë Draug |
Subject: | [Octave-patch-tracker] [patch #8812] Added imquantize to image toolset |
Date: | Thu, 03 Dec 2015 14:27:09 +0000 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0 Iceweasel/38.4.0 |
Update of patch #8812 (project octave): Open/Closed: Open => Closed _______________________________________________________ Follow-up Comment #10: It seems to me that you're reading the arguments in lookup as the same order as imquantize. The only thing that needs to be monotonic is the TABLE, and the TABLE is the imquantize's LEVELS arguments. The call to lookup() has the arguments swapped. I have pushed http://hg.code.sf.net/p/octave/image/rev/90a52e024691 please take a look at the code. I have also extended and corrected your original tests (they were not Matlab compatible). But please ping here with a test case if you find it's not behaving correctly. As far as I could test it, it was correct. _______________________________________________________ Reply to this item at: <http://savannah.gnu.org/patch/?8812> _______________________________________________ Message sent via/by Savannah http://savannah.gnu.org/
[Prev in Thread] | Current Thread | [Next in Thread] |