Message189104
@Stinner:
1. While I agree with your idea of what you have done in test_os, (particularly, for determining if platform is supported or not) there seems to be no reason(AFAIK) to have a shutil for cpu_count. I agree with neologox there.
2. Also I am not comfortable with the idea of having multiple 'implementations' of cpu_count.
"There should be one-- and preferably only one --obvious way to do it."
3. The idea of returning 1 by default does not seem to serve a useful purpose. It should be left to the end-user to decide what needs to be done based on error/actual_value received from system. (+1 to Antoine and nedbat)
For eg,
a. Let's say someone works on scheduling and power managment modules. It is important to know that the platform does not support providing cpu_count() instead of giving 1. This will ensure that they don't go about erroneously setting wrong options for scheduler and/or overclocking the CPU too much(or too little).
b. On the other hand if another user just wants to use a cpu_count number from a his application to determine the number of threads to spawn he can set
th = cpu_count() or 1
(on a side note: *usually* for programs that are non IO intensive and require no/little synchronization it is best to spawn cpu_count() number of threads)
These are just 2 examples to demonstrate that it must be the end-user who decides what to do with the proper_value or reasonable_error_value given by cpu_count()
4. +1 to Antoine on last comment ;-) |
|
Date |
User |
Action |
Args |
2013-05-13 08:15:46 | Yogesh.Chaudhari | set | recipients:
+ Yogesh.Chaudhari, pitrou, vstinner, nedbat, trent, ezio.melotti, r.david.murray, neologix, serhiy.storchaka, kushal.das, dilettant |
2013-05-13 08:15:46 | Yogesh.Chaudhari | set | messageid: <1368432946.86.0.00667887882645.issue17914@psf.upfronthosting.co.za> |
2013-05-13 08:15:46 | Yogesh.Chaudhari | link | issue17914 messages |
2013-05-13 08:15:46 | Yogesh.Chaudhari | create | |
|