[Top][All Lists]

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

[Octave-bug-tracker] [bug #55780] [octave forge] (image) imresize with '

From: Carnë Draug
Subject: [Octave-bug-tracker] [bug #55780] [octave forge] (image) imresize with 'cubic' option is very wrong
Date: Tue, 5 Mar 2019 06:33:45 -0500 (EST)
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Firefox/60.0

Follow-up Comment #7, bug #55780 (project octave):

> Carnë makes a good point, there should always be a comment when
special-casing something.

I have come to find that a test for that special case works best. Comments can
become out of date.

> Is the original author still around to explain this special case?

He's no longer involved with Octave. I would guess that he wouldn't remember
why the special case anyway, it's been 15 years.


Since you're talking about cleaning up imremap, I would advise in investing
your time in implementing imwarp instead. That would be matlab compatible.
imremap is Octave only and is meant to do a  very similar job. Eventually,
imremap should be throw out and replaced with the Matlab compatible function,
so you might as well do that now.


Reply to this item at:


  Message sent via Savannah

reply via email to

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