really, the changes in example were just me playing around with furious,
making sure I understood what it was doing, except for the part of
the ContextIntroHandler where I changed 5 to 55 in:
for i in xrange(5):



On Mon, Jan 14, 2013 at 11:33 AM, Robert Kluin <notifications@github.com>wrote:

> In example/__init__.py:
>
> >
> >          # Create a new furious Context.
> >          with context.new() as ctx:
> >              # "Manually" instantiate and add an Async object to the Context.
> > -            async_task = Async(
> > -                target=example_function, kwargs={'first': 'async'})
> > -            ctx.add(async_task)
> > +            ctx.add(target=whatup, args=[randint(1,20)])#,callbacks={'success': oooeee})
> > +            ctx.add(target=whatup, args=[randint(1,20)])#,callbacks={'success': oooeee})
> > +            ctx.add(target=whatup, args=[randint(1,20)])#,callbacks={'success': oooeee})
> > +            ctx.add(target=whatup, args=[randint(1,20)])#,callbacks={'success': oooeee})
> > +            ctx.add(target=whatup, args=[randint(1,20)])#,callbacks={'success': oooeee})
>
> Was this intended to be committed? If it was, I would vote that we at
> least make this 1) conform to PEP8, including line length, 2) make this
> code a little nicer:
>
> for _ in xrange(20):
>     sync_task = Async(target=whatup, args=[randint(1,20)])
>     ctx.add(async_task)
>
> For callbacks, I think you could use the defaults decorator to clean this
> up too.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/WebFilings/furious/pull/17/files#r2637685>.
>
>