JuliaGPU / JuliaGPU/GPUArrays.jl
Re-add convolution kernel
- 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