<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 21, 2014 at 4:09 AM, Markus Neteler <span dir="ltr"><<a href="mailto:neteler@osgeo.org" target="_blank">neteler@osgeo.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<div><div class="h5"><br>
On Fri, Nov 21, 2014 at 5:00 AM, <<a href="mailto:svn_grass@osgeo.org">svn_grass@osgeo.org</a>> wrote:<br>
> Author: wenzeslaus<br>
> Date: 2014-11-20 20:00:29 -0800 (Thu, 20 Nov 2014)<br>
> New Revision: 62845<br>
><br>
> Added:<br>
> grass/trunk/scripts/d.shadedmap/dshadedmap.png<br>
> grass/trunk/scripts/r.shadedmap/<br>
> grass/trunk/scripts/r.shadedmap/r.shadedmap.html<br>
> grass/trunk/scripts/r.shadedmap/<a href="http://r.shadedmap.py" target="_blank">r.shadedmap.py</a><br>
> grass/trunk/scripts/r.shadedmap/rshadedmap.png<br>
> Removed:<br>
> grass/trunk/scripts/r.shadedmap/d.shadedmap.html<br>
> grass/trunk/scripts/r.shadedmap/<a href="http://d.shadedmap.py" target="_blank">d.shadedmap.py</a><br>
> Modified:<br>
> grass/trunk/scripts/Makefile<br>
> grass/trunk/scripts/d.shadedmap/d.shadedmap.html<br>
> grass/trunk/scripts/d.shadedmap/<a href="http://d.shadedmap.py" target="_blank">d.shadedmap.py</a><br>
> grass/trunk/scripts/r.shadedmap/Makefile<br>
> Log:<br>
> d.shadedmap: split into two modules d.shadedmap for display and r.shadedmap for creating raster<br>
><br>
> * the raster creation functionality was anyway not sharing any code with the display part in the original d.shadedmap<br>
> * similar things are parameters and manual, so there is now some duplication in this<br>
> * thanks to r.shadedmap, GRASS shading is available to QGIS and others<br>
> * in wxGUI one can create the raster in usual way (before: pressing OK or Apply in d command dialog with no overwrite support)<br>
> * adds also brighten functionality for r.shadedmap implemented according to r.his manual (probably not perfect)<br>
> * changing examples to NC and adding images (different image for each module)<br>
<br>
</div></div>Thanks for your efforts, Vaclav.<br>
<br>
But honestly I am not convinced to introduce yet another script<br>
"r.shadedmap" which may add confusion.<br>
At time we have<br>
<br>
- d.his<br>
- d.rgb<br>
- d.shadedmap<br>
- r.his<br>
- r.rgb<br>
- r.blend<br>
- r.shaded.relief<br>
<br>
new<br>
- r.shadedmap<br>
<br>
I see confusion between r.blend and r.shadedmap:<br>
<br>
r.blend: Blends color components of two raster maps by a given ratio.<br>
r.shadedmap: Drapes a color raster over an shaded relief or aspect map.<br>
<br>
and would suggest to merge both into one script.<br>
<br></blockquote><div>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.<br><br></div><div>It is true that the inputs and outputs of r.shadedmap and r.blend are the same but what they are doing is different (at least according to my rough idea about r.blend). Merging would again hide the creating of colorful shaded map in some other module.<br></div><div><br><div>The other reason for splitting d.shadedmap into two
was that in code there were basically two different scripts not sharing
any code. r.shadedmap does not share any code with r.bled and thus there would be still the reason to split them because I think that no shared code points to the fact that the functionalities are not related enough to be in one module and you can better express the difference be keeping them separate.<br></div><div><br></div>I'm not saying that the names or descriptions cannot be improved. I just left them as they are since I had no better idea.<br><br></div><div>Also I forgot about r.bled when I was committing r.shadedmap, they should definitively mention each other in see also. Which reminds me about idea of mentioning differences to given module in see also section.<br><br></div><div>Also I should mention that r.blend is one of the modules I want to change from prefix (output_prefix) to explicit options (red, green, blue). Option to create composite would make sense too.<br><br></div><div>Just partially related is that i.rgb.his and its sister module i.his.rgb are using input and output in their options (red_input, green_input, ..., hue_output, ..., hue_input, ...). This might be actually helpful with understanding what are inputs and what outputs but I actually think that it would be better to remove the _input and _output parts from the option names for consistency with the other modules and also to make them shorter. What is input and what is output should be clear from the name of the module and description of the options. The current long names are not much used anyway and some of the possible shortened form si used instead I'm afraid.<br><br></div><div>I hope you will agree at least with part of this,<br></div><div>Vaclav<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
thanks<br>
Markus<br>
_______________________________________________<br>
grass-dev mailing list<br>
<a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/grass-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-dev</a><br>
</blockquote></div><br></div></div>