[Octave-patch-tracker] [patch #8265] [image package new functions] imgradientxy.m imgradient.m

[Carnë Draug]

Update of patch #8265 (project octave):

                  Status:                    None => In Progress            
             Assigned to:                    None => carandraug             

    _______________________________________________________

Follow-up Comment #1:

Thank you for contribution. I'm quite interested in this functions and would
like to add them to the image package soon, before the new release.

I don't have time to take a look at closely at them yet, I'll do it later. But
first of all, could you please add some tests to the functions? Take a look at
the tests added at these for examples (tests are the %! at the bottom of the
files) and let us know if you have any doubt

* https://sourceforge.net/p/octave/image/ci/default/tree/inst/im2col.m

* https://sourceforge.net/p/octave/image/ci/default/tree/inst/imbothat.m

* https://sourceforge.net/p/octave/image/ci/default/tree/inst/padarray.m

>From quick glance, I only noticed that the help text for imgradientxy is
wrong. Sound like it was written for imgradient though not copy and paste
since you have something else there.

There's also some minor cosmetic issues required to follow the Octave coding
guidelines which I'll gladly apply myself later.

    _______________________________________________________

Reply to this item at:

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

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