[Octave-patch-tracker] [patch #8571] image package: three deconvolution routines (Wiener, RL, RLTV)

[Carnë Draug]

Follow-up Comment #4, patch #8571 (project octave):

Wed 18 Feb 2015 08:35:57 GMT, comment #3:
>> I don't want to end up with 6 deconvolution functions
>> that are very similar but not quite.
>>
>
> I had thought that the problem currently is not that there are too many
deconvolution function in Octave, but that there is none at all.

Yes, that is true now. But adding then now to have them removed one day when
we get a Matlab compatible version, that will create problems.


> If you have a matlab compatible deconvolution implementation however, I
agree that this should be included instead, even though it does not support
regularization for Richardson-Lucy deconvolution, AFAIK.
> 
> It might be best to provide our deconvolution routines not as part of the
image package, but as a separate optional addon, because they are
complementary to what matlab has built in, and are thus also useful for matlab
users. Is there any other repository for octave contributions? 

If there are something these implementations can do, I will try to merge them
into Matlab compatible versions. Kinda like graythresh has an extra option to
select different algorithms.

> Concerning the dimension, I wonder whether there is a way to avoid the
discrimination of two cases, because Octave has different convolution
functions for one and two dimensionc ([fft]conv versus [fft]conv2).

It also has convn which you can use when you want to support N dimensions. The
image package also now has a corresponding fftconvn.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?8571>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/