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

[Vaclav Petras]

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
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20141130/4458b281/attachment.html>