Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

First GPU implementation of DNNs. #199

Closed
wants to merge 33 commits into from

Conversation

simonpf
Copy link
Contributor

@simonpf simonpf commented Jul 17, 2016

No description provided.

@omazapa
Copy link
Contributor

omazapa commented Jul 18, 2016

HI Simon, I saw your code but I am having the next error without cuda installed.
In file included from input_line_12:33:
In file included from /home/ozapatam/Projects/root/compile/include/TMVA/MethodDNN.h:62:
/home/ozapatam/Projects/root/compile/include/TMVA/DNN/Architectures/Cuda.h:23:10: fatal error: 'cuda.h' file not found
#include "cuda.h"
May you need to put some macrodefines there.
Also in the cmake file you are putting the compiler manually set(CUDA_HOST_COMPILER "/usr/bin/g++-4.8") that is not good idea, we can to enable C++ compilation by default using CUDA_HOST_COMPILATION_CPP or are you having other issue there?

* compute block. Contains the elements that will be row-wise accumulated.
*/
//____________________________________________________________________________
__device__ void reduce_sum_vertical(CudaDouble_t *result, CudaDouble_t * sdata);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The coding convention requires camel case for function names.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants