AcademySoftwareFoundation / OpenCue

A render management system you can deploy for visual effects and animation productions.
https://www.opencue.io
Apache License 2.0
832 stars 202 forks source link

All machines under 12 Cores seen as single core machine. #552

Open samkenw opened 4 years ago

samkenw commented 4 years ago

Hi there,

could someone explain to me, why all machines having less than 12 cores (on windows) are seen as single core machines?

https://github.com/AcademySoftwareFoundation/OpenCue/blob/07bca302184e3037e00eee00ee4cdddaace2d637/rqd/rqd/rqmachine.py#L461-L470

` import multiprocessing __totalCores = multiprocessing.cpu_count() * 100 if totalCores > 1200: totalCores = totalCores / 2 numProcs = 2

All other systems will just have one proc/core

    if not __numProcs or not __totalCores:
        __numProcs = 1
        __totalCores = rqconstants.CORE_VALUE`

is this necessary or can it be removed?

Thanks in advance, Sam

gregdenton commented 4 years ago

Hi Sam, This code looks very Sony specific and should likely be removed/refactored to be more generic. We are still working on getting full windows support, so hopefully these types of bugs will be cleaned up soon. Thanks for reporting this! -Greg

samkenw commented 4 years ago

Thanks Greg, that's what I suspected. Can you tell me if there is an eta for full windows support? Greetings Sam

gregdenton commented 4 years ago

Sorry I don't have an eta right now. You can follow along with issue #61 which should get you any updates.

bcipriano commented 4 years ago

@smith1511 Is this resolved now, do you know?

Porges commented 4 years ago

@bcipriano No, the logic has changed but number of actual processors is still not properly detected. There’s no easy way to do this in Python, it will be a Win32-specific solution.