JuliaGPU / JuliaGPU/GPUArrays.jl

Re-add convolution kernel

Open
#117 3 comments 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
Julia
Stars
450
Forks
104
Avg merge
1d 4h
Merged PRs (30d)
10

Description

I have looked at the `convolution!` code but there are some details I am missing.

- Can the kernel have any shape?
- It seems the method returns the same shape of the original but there is no option for padding or not padding.

Wouldn't it be nice to have some documentation and several tests showing how to use it?

I have done some tests:
https://github.com/davidbp/learn_julia/blob/master/GPU_compute/GPUArrays/GPUArrays_convolution.ipynb

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the `convolution!` implementation and the linked GPUArrays convolution notebook. Use the notebook's experiments to determine the supported kernel shapes and padding behavior, then document those behaviors and add tests covering the examples. Done means the usage and shape behavior are documented and the tests pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
julia
Domain
documentation, testing
Issue type
Documentation
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.