Skip to content

Add TornadoVM GPU convolution backend support - #125

Open
Pakhi-7831 wants to merge 1 commit into
sumit3203:masterfrom
Pakhi-7831:pr-1-backend
Open

Add TornadoVM GPU convolution backend support#125
Pakhi-7831 wants to merge 1 commit into
sumit3203:masterfrom
Pakhi-7831:pr-1-backend

Conversation

@Pakhi-7831

Copy link
Copy Markdown
  • Created a new file ConvTornado.java that holds the codes for Tornado-based GPU convolution, implementing Taskgraph and parallel loop running for decreased time.
  • Added function definitions to interface IConv.java for the new functions created for efficient time utilization, with subsequent modification in Conv.java and ConvGpu.java (minimal for this one, to avoid errors as of now)
  • Modified ConvFactory.java for fixing the respective backend routing as well

Comment thread src/dsp/IConv.java

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

not a good idea to include void convolveStructGrad(FloatProcessor src, float[] kernx, float[] kern_diff1,
FloatProcessor gradx, FloatProcessor grady);
void convolveStructSmooth(float[] kernx, float[] kern_diff1,
FloatProcessor gx2, FloatProcessor gy2, FloatProcessor gxy) at the interface level. These are concrete applications.

Comment thread src/dsp/gpu/ConvGpu.java

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

My suggestion is to define a separate application Interface wehre you define the methods and then to inherit both interfaces.

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.

2 participants