[Octave-patch-tracker] [patch #9354] image package: new function wiener2.m

[Avinoam Kalma]

Follow-up Comment #2, patch #9354 (project octave):

Looks very good, passes all the tests, and even has a nice demo.

I can only remark on a few typos:

* Missing space in line 119.
* I would change "formulas" to "equations" in line 46.
* lines 43-46 should be part of the user documentation.
* Minor incompatibility in a corner case: wiener2 (I, [-5 0]) returns zeros in
Matlab, and does not issue an error. I do not know if you should fix this,
because it is really an error.

I recommend to include wiener2 in image package.

Thanks :-)

    _______________________________________________________

Reply to this item at:

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

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