aparapi icon indicating copy to clipboard operation
aparapi copied to clipboard

[BOUNTY $25] Allows multiple kernels to run at the same time

Open freemo opened this issue 7 years ago • 8 comments

Closes issue #62

freemo avatar Oct 23 '17 02:10 freemo

if im reading the travis report right, it is showing just one test failure the one i mentioned for both opencl sdk's. this is pretty good. is there a way to confgure travis to also run the CPU (java threadpool and 'alternative threading') configurations also? this is where i noticed issues with static arrays - not that i understand their purpose in a kernel context unless it is some represntation of a shared memory. but even in that case i would expect some more careful way of managing that between kernel threads. so maybe its better just to rip out the static array support completely, i dunno

automenta avatar Oct 23 '17 03:10 automenta

@automenta travis just provides an environment. It can run JTP or GPU or CPU or whatever the test is configured to run as. So if you create a test that runs under both modes then travis will test under both.

freemo avatar Oct 23 '17 03:10 freemo

@automenta nice ill check out your most recent commits. Is it fully functional and ready in your opinion?

freemo avatar Jan 05 '18 23:01 freemo

it still needs tested in all cases for each device, including both POCL software libs and GPU hardware libs. but if it doesnt pass everything, it's probably just a few exception cases that shouldnt be too much trouble to fix. i havent tested the PR'd SymmetryLabs code but it shouldnt conflict.

this other branch https://github.com/automenta/aparapi/pull/2 i'd like to pull in also though i dont understand what it does yet.

automenta avatar Jan 06 '18 13:01 automenta

@automenta lets chat on gitter when you get a chance

freemo avatar Jan 06 '18 13:01 freemo

@automenta let me know when you think this PR is ready to be reviewed for inclusion.

freemo avatar Feb 11 '18 21:02 freemo

Codecov Report

Merging #68 into master will increase coverage by 1.27%. The diff coverage is 34.93%.

@@             Coverage Diff              @@
##             master      #68      +/-   ##
============================================
+ Coverage     46.93%   48.21%   +1.27%     
- Complexity      849      891      +42     
============================================
  Files            57       57              
  Lines          9390     9191     -199     
  Branches       1527     1487      -40     
============================================
+ Hits           4407     4431      +24     
+ Misses         4540     4325     -215     
+ Partials        443      435       -8

codecov-io avatar Feb 12 '18 05:02 codecov-io

@automenta do you ever think this PR will be ready? I think last we talked you said it was pretty close?

freemo avatar Apr 15 '18 20:04 freemo