[GRASS-dev] [GRASS-SVN] r62845 - in grass/trunk/scripts: . d.shadedmap r.shadedmap

[Vaclav Petras]

On Sun, Nov 30, 2014 at 8:12 PM, Michael Barton <Michael.Barton at asu.edu>
wrote:

>  My idea is that changing r.shaded.relief -> r.relief allows us to use
> r/d.shaded.relief for relief maps that are colored by another variable
> (elevation, land-cover, etc).
>
> I don't think this would be a good idea. r.relief (former r.shaded.relief)
creates "shade from the (terrain) relief". r.shade (former r.shadedmap, and
d.shade) allows to "put the shade on a colored/colorful raster" where both
shade and raster can be anything. Shade can be e.g. aspect, slope or shade
derived from the relief. I know this is probably clear but his was my
thinking.

I didn't though about r.shade and r.relief names before you suggested them
and I'm not 100% satisfied with r.relief because it is in fact not relief
but shade from relief. However, I like the new names because they are
short, there is no problem with using or not using the dot between words
and they are distinct from each other.

 Michael
>            ____________________
>  C. Michael Barton
>  Director, Center for Social Dynamics & Complexity
>  Professor of Anthropology, School of Human Evolution & Social Change
> Head, Graduate Faculty in Complex Adaptive Systems Science
>  Arizona State University
>
>  voice:  480-965-6262 (SHESC), 480-965-8130/727-9746 (CSDC)
> fax: 480-965-7671 (SHESC),  480-727-0709 (CSDC)
>  www: http://www.public.asu.edu/~cmbarton, http://csdc.asu.edu
>
>
>
>  On Nov 30, 2014, at 5:50 PM, grass-dev-request at lists.osgeo.org wrote:
>
>  *From: *Vaclav Petras <wenzeslaus at gmail.com>
>  *Cc: *GRASS developers list <grass-dev at lists.osgeo.org>
>  *To: *Markus Neteler <neteler at osgeo.org>
>  *Date: *November 30, 2014 at 5:31:35 PM MST
>  *Subject: **Re: [GRASS-dev] [GRASS-SVN] r62845 - in grass/trunk/scripts:
> . d.shadedmap r.shadedmap*
>
>
>
>
> On Sat, Nov 29, 2014 at 9:43 AM, Vaclav Petras <wenzeslaus at gmail.com>
> wrote:
>
>>
>>
>>  On Sat, Nov 29, 2014 at 9:30 AM, Markus Neteler <neteler at osgeo.org>
>> wrote:
>>
>>> On Sat, Nov 29, 2014 at 12:37 AM, Vaclav Petras <wenzeslaus at gmail.com>
>>> wrote:
>>> >
>>> > On Fri, Nov 28, 2014 at 5:40 PM, Michael Barton <
>>> Michael.Barton at asu.edu>
>>> > wrote:
>>> >>
>>> >> Probably nothing to be done about this since r.shaded.relief goes
>>> back a
>>> >> long time. But a more sensible renaming would have the script now
>>> called
>>> >> r.chaded.relief renamed to simply r.relief. Then the modules that
>>> drape a
>>> >> colored map onto a relief map could be called x.shaded.relief,
>>> >> x.shadedrelief, or something like that.
>>> >>
>>> > That's an interesting option. I'm not sure if I like it but it's worth
>>> > exploring. If we go even further we get:
>>> >
>>> > r.shaded.relief -> r.relief
>>> > r.shadedmap -> r.shade
>>> > d.shadedmap -> d.shade
>>>
>>> ... along with:
>>> - r.blend
>>>
>>> Yes, I like this list better than the current names of discussed modules.
>>>
>>>   So, if you like it, I can like it too and I will change it hopefully
>> soon. Just to be sure: including r.shaded.relief -> r.relief?
>>
>>
> r.shaded.relief -> r.relief
> r.shadedmap -> r.shade
> d.shadedmap -> d.shade
>
>  Done in r63305 with few other improvements.
>
> Now I'm not sure with [rd].shade parameters. reliefmap and drapemap are
> not particularly explanatory. My idea was shade and color which is better
> when shade=slope compared to relief=slope. color=elevation is also quite
> understandable but of course it is quite unexpected that the color option
> is a raster map.
>
>  There is also one detail I'm not sure about. r.relief might be now
> missing one keyword and that is "shaded relief". "relief" should be there
> too but having also "shaded relief" distinguishes it from things like
> "local relief (model)".
>
> http://trac.osgeo.org/grass/changeset/63305
>
>    Vaclav
>>
>>  Markus
>>>
>>
>>
>
>
>
> _______________________________________________
> grass-dev mailing list
> grass-dev at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/grass-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20141130/8af4ed8b/attachment-0001.html>