modified threader to be more cpu efficient

[Charcol]

Instead of the while loop waiting for a worker to become available then assigning a task, in this structure the workers request tasks when they're free. The original while loop constantly querying the worker queue would cause the python interpreter to lock, delaying the workers going back to the ready pool. So on larger tasks queues with more threads, this would be faster :)

[hakluke]

nice

…

On Fri, Oct 26, 2018 at 12:15 PM Charelle Collett ***@***.***> wrote: Instead of the while loop waiting for a worker to become available then assigning a task, in this structure the workers request tasks when they're free. The original while loop constantly querying the worker queue would cause the python interpreter to lock, delaying the workers going back to the ready pool. So on larger tasks queues with more threads, this would be faster :) ------------------------------ You can view, comment on, or merge this pull request online at: #1 Commit Summary - modified threader to be more cpu efficient File Changes - *M* Interlace/lib/threader.py <https://github.com/codingo/Interlace/pull/1/files#diff-0> (67) Patch Links: - https://github.com/codingo/Interlace/pull/1.patch - https://github.com/codingo/Interlace/pull/1.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ANU_Y6mjSXVQiSaqkvNpTAXb4P70DrWQks5uomIjgaJpZM4X7XfS> .

[codingo]

Makes a lot of sense, thank-you! Going to merge on presumption of testing being done since proj isn't public yet, and will do testing of my own this evening.