Ticket #2640 (closed enhancement: fixed)

Opened 1 year ago

Last modified 9 months ago

inlcude Matt Perry's demtools

Reported by: maphew Assigned to: warmerdam
Priority: normal Milestone: 1.7.0
Component: Utilities Version: unspecified
Severity: normal Keywords: gdaldem
Cc: perrygeo@gmail.com, neteler, jmckenna

Description

adopt Matt Perry's utilities into GDAL to streamline use and maintenance.

svn source: http://perrygeo.googlecode.com/svn/trunk/demtools/

See http://lists.osgeo.org/pipermail/gdal-dev/2008-October/018773.html for related thread.

Change History

10/30/2008 06:34:51 AM changed by maphew

  • component changed from default to Utilities.

(follow-up: ↓ 3 ) 10/30/2008 10:56:46 AM changed by warmerdam

  • cc set to perrygeo@gmail.com.
  • status changed from new to assigned.

I have added Matt to the cc: list.

A few things we need to establish:

1) Is Matt OK with this?

2) If so, is it true that the code is all under the Apache license? Would Matt be willing to relicense it under the GDAL MIT/X license which I believe is similar to the Apache license? This isn't critical but is desirable to keep the GDAL licensing story simple.

3) how will we handle stringtok.h? Would it be acceptable to just incorporate it into each of the programs using it to avoid complexity?

4) Should the names of the utilities be changed? We normally try to avoid very generic names for GDAL utilities that will go into the path. I'd prefer to rename them gdal_aspect, etc.

PS. Should we go ahead with this, I'd also like to propose Matt as a GDAL committer if he is interested so he can maintain the utilities directly in GDAL SVN.

(in reply to: ↑ 2 ) 12/06/2008 11:17:52 AM changed by cbalint

Replying to warmerdam:

3) how will we handle stringtok.h? Would it be acceptable to just incorporate it into each of the programs using it to avoid complexity?

See patch for #2389, it should get rid of stringtok.h and use internal GDAL API.

04/23/2009 06:25:47 PM changed by maphew

I would love to see it happen. Unfortunately I won't have any time to devote it, otherwise I would have done this a long time ago!

You have my full permission to fork and relicense the tools under an X/MIT license - no problem at all.

  • Matt Perry

-- ·´¯·.¸. , . .·´¯·.. ><((((º>·.¸¸.·´¯·.¸.·´¯`·...¸><((((º> "The best way to predict the future is to invent it." -- Alan Kay Matthew T. Perry http://www.perrygeo.net

On Thu, Apr 23, 2009 at 3:12 PM, Matt Wilkie <maphew@gmail.com> wrote:

Hello Matt, I'd love to see your demtools included in the core GDAL package. Frank Warmerdam is open to the idea but has a couple of questions, namely: are you okay with the idea? and are you amenable to changing the license to match GDAL (X/MIT)? The latter is not a showstopper. (see http://trac.osgeo.org/gdal/ticket/2640). thanks for considering it! -- another matt

04/24/2009 01:10:01 AM changed by hobu

ok, I have reworked hillshade to use the C API and I have updated it a little bit. If someone could test trunk to make sure we're doing the right thing, it would be greatly appreciated.

Matt Perry, please check the licensing and copyright http://svn.osgeo.org/gdal/trunk/gdal/apps/gdalhillshade.cpp

Thanks!

r16829

04/24/2009 01:21:16 AM changed by maphew

thanks Hobu; Might want to change "illshade" to "hillshade" :)

04/24/2009 10:49:19 AM changed by hobu

Upon reflection, I think we should congregate these utilities into a single utility -- gdaldem (I'm totally flexible on the name -- and have each be a separate operation within that rather than proliferating five more GDAL utilities.

Their basic operation, open a dataset, fetch a band, create a new dataset/band, process is all the same. I think this approach would ease maintenance in such a way that someone unfamiliar with the algorithms would be able to maintain the housekeeping code easily and wouldn't have to do it in five places. Additionally, it would allow compartmentalizing of the algorithms such that they could be put in the library proper if we had the wherewithal to do so.

I also think Matt Perry should be a GDAL committer. I will propose so on -dev now.

04/24/2009 12:19:44 PM changed by maphew

I like the idea of gdaldem with commands rather than seperate utilities.

04/24/2009 03:31:54 PM changed by neteler

  • cc changed from perrygeo@gmail.com to perrygeo@gmail.com, neteler.

04/24/2009 04:47:21 PM changed by rouault

  • milestone changed from 1.6.1 to 1.7.0.

Various fixes and improvements in r16833 and r16834

04/24/2009 05:19:08 PM changed by rouault

r16834 : Rename gdalhillshade.cpp as gdaldem.cpp to enable more sub-modes to be added

04/24/2009 05:22:58 PM changed by rouault

Grrr. I meant r16837 in my previous comment.

04/24/2009 05:54:06 PM changed by jmckenna

  • cc changed from perrygeo@gmail.com, neteler to perrygeo@gmail.com, neteler, jmckenna.

04/24/2009 05:57:47 PM changed by rouault

'slope' mode added in r16839

04/25/2009 04:41:23 AM changed by rouault

Matt,

I see that the author and copyright holder of color-relief.cpp is Paul Surgeon. I think we also need his explicit authorization to relicence his utility under X/MIT. Would you mind asking him if he agrees, and if so, could he post in that ticket a note similar to yours ? Thanks.

05/02/2009 01:27:55 PM changed by rouault

  • status changed from assigned to closed.
  • resolution set to fixed.

'color-relief' mode added in r16913. This is a clean room implementation, not based on color-relief.cpp.

05/03/2009 10:13:58 AM changed by warmerdam

  • keywords set to gdaldem.

Note, at least parts of Matt's code are apparently directly derived from GRASS GPL code and cannot be included in GDAL due to licensing differences. See #2975 for details.

05/10/2009 04:47:00 PM changed by neteler

For the record, the license issues have been clarified in #2975.