This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author gdr@garethrees.org
Recipients david.lindquist, docs@python, gdr@garethrees.org, larry, rhettinger
Date 2014-02-25.10:55:03
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1393325704.79.0.90476063878.issue20727@psf.upfronthosting.co.za>
In-reply-to
Content
I suspect I messed up the timing I did yesterday, because today I find that 100 isn't large enough, but here's what I found today (in Python 3.3):

    >>> from timeit import timeit
    >>> test = [tuple(range(300))] + [()] * 100
    >>> timeit(lambda:list(roundrobin1(*test)), number=10000) # old recipe
    8.386148632998811
    >>> timeit(lambda:list(roundrobin2(*test)), number=10000) # new recipe
    16.757110453007044

The new recipe is more than twice as slow as the old in this case, and its performance gets relatively worse as you increase the number 300.

I should add that I do recognise that the new recipe is better for nearly all cases (it's simpler as well as faster), but I want to point out an important feature of the old recipe, namely that it discards iterables as they are finished with, giving it worst-case O(n) performance (albeit slow) whereas the new recipe has worst case O(n^2). As we found out with hash tables, worst-case O(n^2) performance can be a problem when inputs are untrusted, so there are use cases where people might legitimately prefer an O(n) solution even if it's a bit slower in common cases.
History
Date User Action Args
2014-02-25 10:55:04gdr@garethrees.orgsetrecipients: + gdr@garethrees.org, rhettinger, larry, docs@python, david.lindquist
2014-02-25 10:55:04gdr@garethrees.orgsetmessageid: <1393325704.79.0.90476063878.issue20727@psf.upfronthosting.co.za>
2014-02-25 10:55:04gdr@garethrees.orglinkissue20727 messages
2014-02-25 10:55:03gdr@garethrees.orgcreate