Fixed

On Oct 2, 2013, at 10:44 AM, Matthew Rocklin <notifications@github.com> wrote:

> In toolz/itertoolz/recipes.py:
> 
> > @@ -16,3 +18,29 @@ def countby(func, seq):
> >          groupby
> >      """
> >      return frequencies(map(func, seq))
> > +
> > +
> > +def partitionby(f, s):
> > +    """ Partition a sequence according to a function
> > +
> > +    Partition `s` into a sequence of lists such that every time `f(s)`
> > +    changes, a new list is started and items are collected into that
> > +    list.
> > +
> > +    Note: mapping `compose(list, second)` is required to make this
> > +    both Python 2 and 3 compatible (Python 3 works without the
> > +    application of `list`).
> This makes it sound like users will need to call compose(list, second) if they want to use this function.
> 
> This sounds like a note for developers, not users. Perhaps it would be better as a comment, not a docstring.
> 
> —
> Reply to this email directly or view it on GitHub.
>