jegonzal / PowerGraph

PowerGraph: A framework for large-scale machine learning and graph computation.
344 stars 518 forks source link

ncpus does not work if NO_OPENMP is true #167

Closed intoraw closed 9 years ago

intoraw commented 9 years ago

In linux, configure graphlab with using --no_openmp running graphlab pagerank like this :

mpiexec -n 5 -hostfile ~/machines /home/zork/Dev-pla/graphlab-trace/release/toolkits/graph_analytics/pagerank \
--graph /home/zork/graph/twitter_rv  \
--format snap --iterations=10 --ncpus=10 

will only use 1 cpu.

dbickson commented 9 years ago

We rely on openmp for running threads in parallel. If openmp is disabled we can not run on more than 1 cpu.

intoraw commented 9 years ago

Some code dost not rely on openmp ( such as synchronous_engine.hpp ), so i think run more than 1 cpu without openmp is fine and it worked ok.
Did i miss anything ?

dbickson commented 9 years ago

I think you are right! I will submit your fix.

Thanks!!

http://www.graphlab.com Danny Bickson Co-Founder US phone: 206-691-8266 Israeli phone: 073-7312889 https://twitter.com/graphlabteam http://www.linkedin.com/company/graphlab https://www.facebook.com/graphlabinc http://www.youtube.com/user/GraphLabInc

On Wed, Nov 12, 2014 at 6:06 PM, Pin Gao notifications@github.com wrote:

Some code dost not rely on openmp ( such as synchronous_engine.hpp ), so i think run more than 1 cpu without openmp is fine and it worked ok.

Did i miss anything ?

— Reply to this email directly or view it on GitHub https://github.com/graphlab-code/graphlab/issues/167#issuecomment-62742472 .