Message 188906 - Python tracker

➜

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	neologix
Recipients	Yogesh.Chaudhari, ezio.melotti, kushal.das, nedbat, neologix, pitrou, r.david.murray, trent, vstinner
Date	2013-05-11.11:50:08
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<CAH_1eM3W5vihSWyWcmtgUj9w=a9Em2wD0w69OGQN+hz7u7WKOQ@mail.gmail.com>
In-reply-to	<[email protected]>

Content
> Based on the conversation and the particular inputs to the thread form neologix and ezio, I would like to submit this patch. > > It probably needs modification(s) as I am not sure what to do with the implementation that is already present in multiprocessing. This patch simply calls the os.cpu_count() from multiprocessing now and behaves as it would have previously. Thanks, but it would be better to reuse Trent's C implementation instead of multiprocessing's: http://hg.python.org/sandbox/trent/file/dd1c2fd3aa31/Modules/posixmodule.c#l10213

> Based on the conversation and the particular inputs to the thread form neologix and ezio, I would like to submit this patch.
>
> It probably needs modification(s) as I am not sure what to do with the implementation that is already present in multiprocessing. This patch simply calls the os.cpu_count() from multiprocessing now and behaves as it would have previously.

Thanks, but it would be better to reuse Trent's C implementation
instead of multiprocessing's:
http://hg.python.org/sandbox/trent/file/dd1c2fd3aa31/Modules/posixmodule.c#l10213

History
Date	User	Action	Args
2013-05-11 11:50:08	neologix	set	recipients: + neologix, pitrou, vstinner, nedbat, trent, ezio.melotti, r.david.murray, kushal.das, Yogesh.Chaudhari
2013-05-11 11:50:08	neologix	link	issue17914 messages
2013-05-11 11:50:08	neologix	create