[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #8377] [octave forge] (mapping) 3D Coordin
|
From: |
Philip Nienhuis |
|
Subject: |
[Octave-patch-tracker] [patch #8377] [octave forge] (mapping) 3D Coordinate system conversion functions for Geodesy |
|
Date: |
Fri, 22 Mar 2019 16:22:20 -0400 (EDT) |
|
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0 SeaMonkey/2.48 |
Follow-up Comment #23, patch #8377 (project octave):
Looks very good!
As I'm a nitpicker, apologies for that :-) , I couldn't help finding a few
cosmetic style things (a.o., please always use double ## before comment lines;
lines please max. 80 chars except @deftypefn lines); I'll handle those for
aer2enu.m.
Same guy as patch 9709, 9720, 9736 and 9756?
Check what's in the mapping repo to avoid implementing already existing
functions:
http://hg.code.sf.net/p/octave/mapping/file/71b3081e51d1/inst
BTW it looks like a new mapping release may be coming soon now; mapping works
fine with new geometry and matgeom (I have test versions of those running),
their maintainer is always very busy however. Once they are on the package
release tracker I'll make a new mapping release.
So there's still time for new functions.
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/patch/?8377>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/