gpu.js icon indicating copy to clipboard operation
gpu.js copied to clipboard

Chain combineKernels?

Open pan-long opened this issue 7 years ago • 8 comments

When I try to pass a combinedKernel into combineKernels() to chain them, I got this:

Uncaught TypeError: arguments[i].setCanvas is not a function at GPU.combineKernels

pan-long avatar Nov 15 '17 13:11 pan-long

Can we getore context, like example code?

robertleeplummerjr avatar Nov 15 '17 14:11 robertleeplummerjr

I'm using GPU.js to run bellman Ford algorithm, code here: https://github.com/pan-long/SSSP-on-GPU/blob/master/bellman_ford_gpu.js

I have to have a loop to keep running bellmanFordKernel, it's now slower than Dijkstra running on GPU. I'm wondering is it because of performance penalty brought up by keep transferring data between CPU and GPU. Then I found out combineKernels, but it doesnt support chaining.

On Wed, Nov 15, 2017, 10:26 PM Robert Plummer [email protected] wrote:

Can we getore context, like example code?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gpujs/gpu.js/issues/224#issuecomment-344608080, or mute the thread https://github.com/notifications/unsubscribe-auth/AD3olvFKMYsaPtJ-dLW009Vo-B6KJ3BNks5s2vSsgaJpZM4Qe9Ds .

pan-long avatar Nov 15 '17 14:11 pan-long

I think you are dead right. We need to add this as a feature, so we can chain values through. Give me some time this week and I'll add support. We'll get this thing faster!

robertleeplummerjr avatar Nov 15 '17 16:11 robertleeplummerjr

Can we have access to that code again? Or something simple that outlines the chaining being done? Clarity helps us help you! 😉

robertleeplummerjr avatar Nov 18 '17 02:11 robertleeplummerjr

Hi, I have made it public. Thanks for your help!

On Sat, Nov 18, 2017, 10:05 AM Robert Plummer [email protected] wrote:

Can we have access to that code again? Or something simple that outlines the chaining being done? Clarity helps us help you! 😉

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gpujs/gpu.js/issues/224#issuecomment-345410426, or mute the thread https://github.com/notifications/unsubscribe-auth/AD3oltX5oiYOhVceIskfoVqVrrOzenllks5s3jtlgaJpZM4Qe9Ds .

pan-long avatar Nov 18 '17 04:11 pan-long

Been thinking about this over the weekend, I will have a solution soon, fyi.

robertleeplummerjr avatar Dec 04 '17 15:12 robertleeplummerjr

This needs clearer direction. Can you give a non-working jsfiddle?

robertleeplummerjr avatar May 22 '19 22:05 robertleeplummerjr

I have run into this bug... Great library btw!

fgatti675 avatar Jan 13 '21 11:01 fgatti675