-
Notifications
You must be signed in to change notification settings - Fork 75
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
1D Convolution example #2220
1D Convolution example #2220
Conversation
6e789a4
to
8286036
Compare
@mehmetyusufoglu A hint, you can use a draft instead a pull request, if you nobody should merge to early. There is option on the right corner to covert the PR to a draft. |
34039a1
to
afb11ce
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here is some feedback for a start.
11ca997
to
3901fcc
Compare
acaaf9e
to
c7665f3
Compare
39546d8
to
a531e69
Compare
nativeOutputDeviceMemory, | ||
inputSize, | ||
filterSize); | ||
alpaka::wait(queue); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no wait
is required because the next operation is in the same queue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok I added wait after the second operation. Thanks a lot.
6025f69
to
1920952
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last piece of feedback. The rest looks good.
Please squash all commits as well!
d844f28
to
7343be9
Compare
7343be9
to
d37cad5
Compare
This pull request consisting 1D convolution implementation is an exercise for a 2D or 3D convolution example.