[GRASS-dev] [GRASS-SVN] r62845 - in grass/trunk/scripts: . d.shadedmap r.shadedmap
neteler at osgeo.org
Thu Nov 27 09:12:30 PST 2014
On Fri, Nov 21, 2014 at 4:15 PM, Vaclav Petras <wenzeslaus at gmail.com> wrote:
> On Fri, Nov 21, 2014 at 4:09 AM, Markus Neteler <neteler at osgeo.org> wrote:
>> On Fri, Nov 21, 2014 at 5:00 AM, <svn_grass at osgeo.org> wrote:
>> > Author: wenzeslaus
>> > Date: 2014-11-20 20:00:29 -0800 (Thu, 20 Nov 2014)
>> > New Revision: 62845
>> > Added:
>> > grass/trunk/scripts/d.shadedmap/dshadedmap.png
>> > grass/trunk/scripts/r.shadedmap/
>> > grass/trunk/scripts/r.shadedmap/r.shadedmap.html
>> > grass/trunk/scripts/r.shadedmap/r.shadedmap.py
>> > grass/trunk/scripts/r.shadedmap/rshadedmap.png
>> > Removed:
>> > grass/trunk/scripts/r.shadedmap/d.shadedmap.html
>> > grass/trunk/scripts/r.shadedmap/d.shadedmap.py
>> > Modified:
>> > grass/trunk/scripts/Makefile
>> > grass/trunk/scripts/d.shadedmap/d.shadedmap.html
>> > grass/trunk/scripts/d.shadedmap/d.shadedmap.py
>> > grass/trunk/scripts/r.shadedmap/Makefile
>> > Log:
>> > d.shadedmap: split into two modules d.shadedmap for display and
>> > r.shadedmap for creating raster
>> > * the raster creation functionality was anyway not sharing any code
>> > with the display part in the original d.shadedmap
>> > * similar things are parameters and manual, so there is now some
>> > duplication in this
>> > * thanks to r.shadedmap, GRASS shading is available to QGIS and others
>> > * in wxGUI one can create the raster in usual way (before: pressing OK
>> > or Apply in d command dialog with no overwrite support)
>> > * adds also brighten functionality for r.shadedmap implemented
>> > according to r.his manual (probably not perfect)
>> > * changing examples to NC and adding images (different image for each
>> > module)
>> Thanks for your efforts, Vaclav.
>> But honestly I am not convinced to introduce yet another script
>> "r.shadedmap" which may add confusion.
>> At time we have
>> - d.his
>> - d.rgb
>> - d.shadedmap
>> - r.his
>> - r.rgb
>> - r.blend
>> - r.shaded.relief
>> - r.shadedmap
>> I see confusion between r.blend and r.shadedmap:
>> r.blend: Blends color components of two raster maps by a given ratio.
>> r.shadedmap: Drapes a color raster over an shaded relief or aspect map.
>> and would suggest to merge both into one script.
> I don't think that it is a good idea. There are another sources of confusion
> too. d.shadedmap gives nice picture but cannot create the shade itself, so
> you have to find some other module. If you hit r.shaded.relief first, you
> might be confused from getting the shade but not the nice picture.
> r.shadedmap does not changes this two-step approach but at least it makes
> clear how to create shaded map with colors.
Then it should be renamed.
We cannot add continuously new scripts with similar names...
More information about the grass-dev