Conversation
|
Yeah, I'm pretty happy with that, I'd hate to have to tweak this somehow if something were to change on that front. Please ping us when 1.0 is released so we can merge this :) Thanks for the comments on previous checks before. Shouldn't those checks be included in your library too? |
Not really because this is really specific to Psalm, although I admit I don't know the why. For any of those items the library does work though and so would spawning several processes |
|
ok! Looking at the comments, I thought this was a common known issue with multi threading |
|
Should be good; there is now a distinction possible between physical and logical cores. The default picked is logical (as PHP source does and as was done previously here). If you fancy checking out what finder finds what, to find a bug or play around with the order or something, I can recommend those commands |
|
Thanks! |
See theofidry/cpu-core-counter#11 for the motivations.
The code should be sensibly the same, the notable differences are:
NumberOfCpuCoreNotFoundis thrown instead ofLogicException(can easily be changed)hw.ncpuI would also wait on 1.x before merging this if you're happy with it. Technically the current version is fine, I'm just looking for a round of feedback before publishing 1.0.