patchGNU Octave - Patches: patch #9302, [octave forge] (mapping) Included...

 
 

patch #9302: [octave forge] (mapping) Included some functions and improved few others.

Submitter:  Mithil <mithil>
Submitted:  Sat 25 Mar 2017 08:48:06 AM UTC
   
 
Category:  Forge : new function Priority:  3 - Low
Status:  Wont Do Privacy:  Public
Assigned to:  philipnienhuis Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Sun 27 Jan 2019 04:07:26 PM UTC, comment #7: 

All functions here have been provided in other patch reports in the last year.

Closing with "Won't do"

Philip Nienhuis <philipnienhuis>
Group Member
Fri 23 Jun 2017 06:58:56 PM UTC, comment #6: 

No anwer forovalmost 2,5 months.

Lowering priority & postponed

Philip Nienhuis <philipnienhuis>
Group Member
Wed 29 Mar 2017 06:54:35 PM UTC, comment #5: 

Mithil,

Thanks for your contributions.

I took a brief look and noted following:

deg2sm.m  &  deg2nm.m  &  rad2nm.m  &  rad2sm.m:

  • (C) 2013 CarnĂ« Draug ?? (did you copy verbatim from another function? I suppose you as author have the copyright)
  • These functions need tests, also input validation tests. E.g.,


%!error <validatestring: 'feet' does not match any of> earthRadius ("feet")

(and why no "feet" or "nm"/"nauticalmile" or other measures as acceptable input? Octave could do better than Matlab!

  • The persistent statement had better be in the first lines of the function body
  • Could you please also implement alternative radii of celestial bodies? E.g., sm2deg has them.


earthradius.m:

  • (C) 2017 CarnĂ« Draug ??  see above
  • needs input validation tests (see example above)


Other than that you did a good job of adhering to coding style standards.

Philip Nienhuis <philipnienhuis>
Group Member
Mon 27 Mar 2017 05:25:37 PM UTC, comment #4: 

BTW have you uploaded the patches for patch #9301 here also, rather than there? In that case I'll close patch #9301

Philip Nienhuis <philipnienhuis>
Group Member
Mon 27 Mar 2017 05:24:28 PM UTC, comment #3: 

Thanks Mithil, I'll take a look.

2nd half of the week I'll have time for this (I hope)

Philip Nienhuis <philipnienhuis>
Group Member
Mon 27 Mar 2017 05:14:31 AM UTC, comment #2: 

Thanks for correcting me.

I have now added the required m-script files and the PATCH file. Looking forward to contributing more.

Mithil <mithil>
Sun 26 Mar 2017 08:09:36 AM UTC, comment #1: 

Thank you.
Please see my comment at patch #9301.
Mainly upload separate files, not one big thing just called "FILE"

Philip Nienhuis <philipnienhuis>
Group Member
Sat 25 Mar 2017 08:48:06 AM UTC, original submission:  


Mithil <mithil>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #40165:  rad2sm.m added by mithil (2KiB - text/x-objcsrc)
file #40161:  earthRadius.m added by mithil (2KiB - text/x-objcsrc)
file #40162:  deg2nm.m added by mithil (2KiB - text/x-objcsrc)
file #40163:  rad2nm.m added by mithil (2KiB - text/x-objcsrc)
file #40164:  deg2sm.m added by mithil (2KiB - text/x-objcsrc)
file #40126:  FILE added by mithil (26KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by siko1056 (Updated the item)
  • -email is unavailable- added by mithil (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 12 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-01-27 philipnienhuis StatusPostponed Wont Do
        Open/ClosedOpen Closed
    2017-10-08 siko1056 SummaryMapping package. Included some functions and improved few others. [octave forge] (mapping) Included some functions and improved few others.
    2017-06-23 philipnienhuis Priority5 - Normal 3 - Low
        StatusNone Postponed
    2017-03-27 philipnienhuis Assigned toNone philipnienhuis
    2017-03-27 mithil Attached File- Added rad2sm.m, #40165
    2017-03-27 mithil Attached File- Added earthRadius.m, #40161
        Attached File- Added deg2nm.m, #40162
        Attached File- Added rad2nm.m, #40163
        Attached File- Added deg2sm.m, #40164
    2017-03-25 mithil Attached File- Added FILE, #40126

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code