Path: csiph.com!usenet.pasdenom.info!gegeweb.org!de-l.enfer-du-nord.net!feeder2.enfer-du-nord.net!feeder1.enfer-du-nord.net!cs.uu.nl!news.stack.nl!newsfeed.xs4all.nl!newsfeed2.news.xs4all.nl!xs4all!post.news.xs4all.nl!not-for-mail
To: python-list@python.org
From: Terry Jan Reedy <tjreedy@udel.edu>
Subject: Re: Idiomatic Python for incrementing pairs
Date: Sat, 08 Jun 2013 13:29:21 -0400
References: <20130607213239.3e39a448@bigbox.christie.dr> <BLU176-W41787828AF1636314057B8D79A0@phx.gbl> <20130607231622.31cb120b@bigbox.christie.dr>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130509 Thunderbird/17.0.6
In-Reply-To: <20130607231622.31cb120b@bigbox.christie.dr>
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.2889.1370712552.3114.python-list@python.org>
Lines: 63
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:47392

On 6/8/2013 12:16 AM, Tim Chase wrote:
> On 2013-06-08 07:04, Carlos Nepomuceno wrote:
>> alpha, beta = (1 if some_calculation(params) else 0, 1 if
>> other_calculation(params) else 0)
>
> This one sets them to absolute values, rather than the incrementing
> functionality in question:
>
>>>    alpha += temp_a
>>>    beta += temp_b
>
> The actual code in question does the initialization outside a loop:
>
>    alphas_updated = betas_updated = 0
>    for thing in bunch_of_things:
>      a, b = process(thing)
>      alphas_updated += a
>      betas_updated += b

I am pretty sure that this is the faster way to get the result.

> and it just bugs me as being a little warty for having temp
> variables when Python does things like tuple-unpacking so elegantly.

So use it ;-).

> That said, as mentioned in a contemporaneous reply to Jason, I haven't
> found anything better that is still readable.

The generalization of what you want is a mutable vector with an in-place 
augmented assignment version of element by element addition. That 
probably has been written. Nnmpy arrays do vector addition and maybe do 
it in-place also (I just do ot know). But here is a custom class for the 
problem you presented.

def process(val):
     return val, 2*val

class Pair(list):
     def __init__(self, a, b):
         self.append(a)
         self.append(b)
     def inc(self, a, b):
         self[0] += a
         self[1] += b

pair = Pair(0, 0)
for val in [1,2,3]:
     pair.inc(*process(val))

print(pair)
 >>>
[6, 12]

This is cleaner but a bit slower than your in-lined version. I did not 
use __iadd__ and += because unpacking 'other' (here the process return) 
in the call does the error checking ('exactly two values') for 'free'.

--
Terry Jan Reedy