On Sun, Mar 22, 2015 at 10:40 AM, Nikos Alexandris <n...@nikosalexandris.net> wrote:
> Vaclav Petras wrote: > > It would be good if you can look to r.sun which other options you can >>>> >>>>> include. >>>>> >>>> > Nikos Alexandris wrote: > > I am. I am also renaming stuff, eg slope_in to slope. I guess this is >>>> wanted. >>>> >>> > VP: > > It should be slope (and slope_value) in the options but I think that's >>> already true. How the variables in the script are named is not as >>> important. It's nice when they are consistent with inputs but there can >>> be >>> some reasons for the opposite. >>> >> > NA: > > If there are reasons, in this, case, please share. > > There is none unless you hit some issue or something. > Also: is the >> intention to have one script for both G7x and G64x? > > It was just simpler at the time of writing. Now there is probably no need to have G6 support since G7 is out. > I would like to >> know, while I am at it, to structure appropriately the addition of >> "linke". >> > > The script works for me, with support for linke now. Will share later >> my working status. >> > > Modified script attached, Please share also the diff. > support only for G7. > > If you will not add G6 support then remove all G7/G6 related if statements. Nikos > _______________________________________________ > grass-dev mailing list > grass-dev@lists.osgeo.org > http://lists.osgeo.org/mailman/listinfo/grass-dev >
_______________________________________________ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev