Brian Sutherland jinty

jinty commented on pull request vanguardistas/mp.importer#1
@jinty

This function is supposed to be passed in as a parameter to "run_in_batches" as the user of this API is meant to define it.

jinty commented on pull request vanguardistas/mp.importer#1
@jinty

I don't think you use "count" anywhere.

jinty commented on pull request vanguardistas/mp.importer#1
@jinty

I am not convinced that it is useful to test different kinds of iterables. The iterable interface in python is very well defined and any code that …

jinty commented on pull request vanguardistas/mp.importer#1
@jinty

why not pass the parameters in as defaults? like: def make_test_context(batch_size=2, start_batch=0): then when you need different values, you can …

jinty commented on pull request vanguardistas/mp.importer#1
@jinty

if the number of results is large, storing them all in a list is probably going to cause memory problems. Perhaps you might think about returning a…

jinty commented on pull request vanguardistas/mp.importer#1
@jinty

len() always returns an int, you should never need to cast here This code only works on a subset of iterables, i.e. those iterables which are "rew…

jinty commented on pull request zopefoundation/zc.resumelb#12
@jinty

Perhaps you mean "uncaught application exceptions"?

@jinty
Failure with plural forms on python 3
jinty commented on pull request zopefoundation/zc.resumelb#3
@jinty

On Wed, Feb 11, 2015 at 01:43:10PM -0800, Jim Fulton wrote: :) BTW, I'm working on a resume-less pool that just selects workers with minimum backl…

jinty commented on pull request zopefoundation/zc.resumelb#3
@jinty

no worries, thanks for zc.resumelb! I prefer slow acceptance of pull requests than bad software any day of the week.

jinty commented on pull request Pylons/deform#251
@jinty

Perhaps I should have used btn-link as an example rather than btn-danger. Here's an example of what happens to the other buttons when you have an e…

jinty opened pull request Pylons/deform#251
@jinty
Have the button css_class override the default button class
1 commit with 10 additions and 6 deletions
jinty pushed to master at jinty/deform
@jinty
jinty pushed to master at jinty/deform
@jinty
jinty commented on issue Pylons/pyramid#1482
@jinty

The management of the connections is more a concern of the WSGI server, not so much of the WSGI application (i.e. Pyramid). Perhaps you could try a…

jinty commented on issue Pylons/pyramid#1482
@jinty

I don't think this can be debugged with the information your provide. Could you provide more information about your setup/code? Ideally, you could …