On Fri, Mar 20, 2015 at 4:41 PM, Nikos Alexandris <n...@nikosalexandris.net> wrote:
> On 20.03.2015 22:27, Vaclav Petras wrote: > >> Hi Nikos, >> >> On Fri, Mar 20, 2015 at 4:07 PM, Nikos Alexandris < >> n...@nikosalexandris.net> >> wrote: >> >> Hi list, @Anna, @Vaclac >>> >>> is there interest to add support for the Linke Turbidity (as an input) to >>> the r.sun.daily script? >>> >>> there certainly is interest is that (although not from me :-). I just >>> >> don't have time to do it. You might be able to add it there. I can do the >> code review then. >> > > On-sight, looks as easy as adding another parameter update along with the > necessary additions which an update like this will require to the rest of > the script. > > There is some simple test (pre-dating testing framework), so you can also try to update that. At least add there the new option combinations you will add. > If I retouch the script, may I replace globally, wherever applicable, the > "%s" % var stuff with '{s}'.format(s=var} ? > > Yes please! My approach (usually) is to use %s when there is one item and format() for more items. But generally, format() seems the best choice. > Nikos >
_______________________________________________ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev