[Octave-patch-tracker] [patch #8320] [mapping package] Conversion between units of length

[Carnë Draug]

Follow-up Comment #1, patch #8320 (project octave):

At a first glance, I don't see anything wrong with the patch.

I believe you are an applicant to GSoC trying to get acquainted with Octave by
taking items from the Short projects page, so I'd suggest you could prepare
two more csets on top of this one.

One to change the NEWS and INDEX file to make reference of the new functions.
See for example, csets 92dd0071c58d and f7c6d8890730, that also add new
functions to this files.

On other to add a few extra tests to convert values in a vector and ND matrix.
Something like:


%!test
%! km = [1.8520    3.7040    5.5560    7.4080    9.2600   11.1120];
%! nm = [1 2 3 4 5 6];
%! assert (km2nm (km), nm)
%! km = reshape (km, [1 3 2])
%! nm = reshape (nm, [1 3 2])
%! assert (km2nm (km), nm)


Of course, any more tests you can think of are welcome.

    _______________________________________________________

Reply to this item at:

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

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