start parallel RANLUX - Githubissues

maedoc / libtvb

TVB C library

6 stars 8 forks source link

start parallel RANLUX #144

Closed AbheekG closed 8 years ago

AbheekG commented 8 years ago

Started working on RANLUX implementation using OpenCL. Very initial stage. Decide on:

seeding: using the legacy initialization or 64-bit xorshift PRNG by George Marsaglia.
using non-parallel implementation for generating one random value
using double.
any other general directions for implementation

todo, may buggy

[x] import ranluxcl code
[x] fix kernel code.
[x] add host code
[x] add api
[x] fill rng array
[ ] add tests
[ ] properly saving state, for later use
[ ] support non-kernel generation for one value
[ ] allow kernel memory for later use
[ ] benchmark
[ ] output to file

paramhanji commented 8 years ago

The build error indicates that CL/cl.h is missing. Perhaps this file need to be added. @abheekg Does this compile properly on your local PC?

maedoc commented 8 years ago

We have to see how to handle OpenCL code on Travis. There should be a package we can add to .travis.yml to get the headers ocl-dev or something. Running actual tests is another story.

maedoc commented 8 years ago

@abheekg rebase on my master branch and push again, that should pull in the header files for OpenCL on Travis.

AbheekG commented 8 years ago

@maedoc Usual programs are able to detect opencl structures and functions by adding only -lOpenCL in gcc. In this, I tried -lOpenCL as well as something extra (in the makefile), but doesn't work even on my PC. Please have a look on makefile. Fixing this will help working on feature faster.

maedoc commented 8 years ago

@abheekg the makefile syntax $AMDAPPSDK isn't correct, it should be $(AMDAPPSDK). I'll keep looking on Monday.

AbheekG commented 8 years ago

@maedoc I modified the makefile and it compiles perfectly on my PC. See, you after weekends.

maedoc commented 8 years ago

@abheekg it looks good :+1: I am setting up my machine for OpenCL development and will try to run it as soon as possible as well as fixing the Travis build.

My comments are mainly stylistic, in order to keep the library organized consistently, and I'll be able to help with that.