patchGNU Octave - Patches: patch #8513, [octave forge] (mapping): SRTM /...

 
 

patch #8513: [octave forge] (mapping): SRTM / .hgt tools

Submitter:  None
Submitted:  Wed 13 Aug 2014 06:43:26 PM UTC
   
 
Category:  Forge : new function Priority:  1 - Later
Status:  None Privacy:  Public
Assigned to:  None Originator Email:  -email is unavailable-
Open/Closed:  Open
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Sun 19 Jan 2020 12:31:37 PM UTC, comment #8: 

OK, I had a brief look.

The functionality isn't very hard to understand for an earth scientist like me, and I think in itself these functions would be a worthwhile addition to the mapping package.

OTOH there's quite a bit of work required w.r.t. Matlab compatibility, style, copyright (what's the contributor's name?), proper texinfo header, input validation (none at all now AFAICS) and BIST tests (also none at all), to mention just a few things.

And next, to link them to other mapping package functions like mapshow, ....

If I pick up at all, it'll be quite a bit into the future.

Priority ==> Later
Title adapted

Philip Nienhuis <philipnienhuis>
Group Member
Sat 18 Jan 2020 06:14:43 PM UTC, comment #7: 

No hurry and no worries Philip.

All comments marked as "Review for Octave 6" means that the patch tracker indicated that they are suitable for Octave core.

Thus I am very happy to delegate them away and there is no need to finish for the Octave 6 release.

Actually 82 patches had this label.  Only 46 remain now.  Many patches simply had a wrong label like this or were outdated, ...

Kai Torben Ohlhus <siko1056>
Group Member
Sat 18 Jan 2020 06:09:44 PM UTC, comment #6: 

(mapping pkg maintainer here)
I'm about to upload a new mapping release that has been ready for a looong time and was only waiting all along for a new geometry package release.

These patches (that I never noted before) probably won't make it into mapping-1.4.0, sorry, but I'll review them.

Philip Nienhuis <philipnienhuis>
Group Member
Sat 18 Jan 2020 11:39:23 AM UTC, comment #5: 

Review for Octave 6.

As mentioned in comment 1, maybe better suited for a domain expert to include these functions into the mapping package

https://octave.sourceforge.io/mapping/index.html

Adding package maintainer to this item.

Kai Torben Ohlhus <siko1056>
Group Member
Tue 19 Aug 2014 10:32:41 PM UTC, comment #4: 

You are right, the name of the mission is SRTM. The format in which the SRTM data is mainly distributed is referred as SRTM format or hgt format. The common file extension for this format is .hgt.

The function srtm in srtm.m reads a .hgt file and delivers a matrix containing its data.

On the other hand, I have found some interesting functions for mapping distributed under BSD license in this site:

http://www.ipgp.fr/~beaudu/matlab.html#SRTM

Anonymous
Sun 17 Aug 2014 06:33:54 PM UTC, comment #3: 


> You can find more information about SRTM format in the following link:
> https://en.wikipedia.org/wiki/Shuttle_Radar_Topography_Mission


This link suggests that that there is no SRTM format, that is the name of the
mission (and data that the mission collected). It does mention however that
it's mainly distributed as hgt file format

> The elevation models derived from the SRTM data are used in Geographic
> Information Systems. They can be downloaded freely over the Internet, and
> their file format (.hgt) is supported by several software developments.


However the external links of that page suggest that you can have the
data on many other file formats. Is your implementation supposed to read
hft files? Because it seems to simple to handle that format.

Carnë Draug <carandraug>
Group Member
Sat 16 Aug 2014 01:52:07 PM UTC, comment #2: 

I have read the documentation of these Matlab functions and I think I can make the functions I wrote compatible with them.

The main differences are that Matlab functions accept geographic coordinates as inputs while the functions I wrote accept matrix indices directly. Obviously, the conversion from geographic coordinates to matrix indices is done by means of an additional argument.

However, coverage function provides more information than viewshed, because the first one provides the clearance in the path while the last one provides boolean data of visibility.

You can find more information about SRTM format in the following link:
https://en.wikipedia.org/wiki/Shuttle_Radar_Topography_Mission

Anonymous
Fri 15 Aug 2014 04:44:40 PM UTC, comment #1: 

As mentioned on the mailing list, this set of functions would belong to the mapping toolbox which is currently unmaintained. Also, a Matlab compatible version of this functions is preferable.

  • profile - could this be implemented to be compatible with mapprofile() ?


  • coverage - could this be implemented to be compatible with los2() or viewshed() ?


It seems to me that your functions do part of those Matlab functions. You wouldn't have to implement all of the possible ways to call those functions (each of them has many), only the methods from the functions you already wrote.

For example, Matlab's mapprofile() gets the data from the current figure if nargin is 0. You can just give an error in that case saying that interactive usage is not yet implemented.

About the STRM files I can't find any information about this file format.

Carnë Draug <carandraug>
Group Member
Wed 13 Aug 2014 06:43:26 PM UTC, original submission:  

A set of tools for handling digital elevation models in SRTM format with GNU Octave.

This patch includes the following files:

  • srtm3.m
  • profile.m
  • coverage.m
  • mat2stl.m


Providing the following functions respectively:

  • srtm3, load STRM3 DEM files
  • profile, calculate terrain profiles
  • coverage, calculate coverage maps
  • mat2stl, convert a matrix to STL surface


You can get more information by reading the comments in each file.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #31897:  SRTM3.tar.gz added by None (1KiB - application/gzip - SRTM3 tools)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by siko1056 (Posted a comment)
  • -email is unavailable- added by siko1056 (OF mapping maintainer.)
  • -email is unavailable- added by None (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 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-01-19 philipnienhuis Priority5 - Normal 1 - Later
        Summary[octave forge] (mapping): SRTM3 tools [octave forge] (mapping): SRTM / .hgt tools
    2020-01-18 siko1056 CategoryNone Forge : new function
        SummarySRTM3 tools [octave forge] (mapping): SRTM3 tools
        Carbon-Copy- Added -email is unavailable-
    2014-08-13 None Attached File- Added SRTM3.tar.gz, #31897

    Back to the top

    Powered by Savane 3.14-3b9d.
    Corresponding source code