[GRASS-dev] Linke turbidity in r.sun.daily

[Vaclav Petras]

On Fri, Mar 20, 2015 at 4:41 PM, Nikos Alexandris <nik at 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 <
>> nik at 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20150320/564821f4/attachment.html>