caffe_train icon indicating copy to clipboard operation
caffe_train copied to clipboard

Published 20 hours ago • verified publisher icon CMU-Perceptual-Computing-Lab

update caffe with the latest master version from BLVC

Open xizero00 opened this issue 6 years ago • 2 comments

Hi all: I have update the caffe with the latest master (2017/7/26) version from BLVC. Would you please merge it? Thanks. Feng

xizero00 avatar Jul 26 '17 07:07 xizero00

EDITED: sorry, I confused the repo. I'll check it and merge it if everything works properly. Thanks!

gineshidalgo99 avatar Jul 26 '17 07:07 gineshidalgo99

I have a main concern...

Caffe removed the data_reader.hpp and data_reader.cpp files because they started to use instead the NCCL library.

However, for what I could understand after taking a look to your code, you have simply copied again the old data_reader files instead of adapting the CPM files to use the new NCCL library as Caffe does, right?

Since then we are using two different methods to parallelize the GPUs (the one the CPM files use vs. the one everything else in Caffe uses), is this completely thread safe or even single-thread safe? I am asking because I am not an expert in NCCL not Caffe data_reader, but keep using both sounds to me like a bug-prone idea...

gineshidalgo99 avatar Aug 21 '17 23:08 gineshidalgo99