search

hughperkins / clnn

OpenCL backend for Torch nn neural networks library
BSD 2-Clause "Simplified" License
126 stars 16 forks source link

SpatialFullConvolution in clnn #44

Open bionick87 opened 7 years ago

bionick87 commented 7 years ago

Hi,

I want to ask you if is possible to convert SpatialFullConvolution for using it in clnn.

Best,

Nick

ViliusT commented 7 years ago

Hey @bionick87, if you look at #42 you will see that I've encountered this issue as well, and was underway to port SpatialFullConvolution - the only issue was that distro-cl is forked just before spatialFullConvolution was commited to torch;

I started the merge from upstream, but then a couple of Nvidia cards came over and... yeah, never finished it. If you're up for the task, feel free to review the referenced thread - if I remember correctly, https://github.com/torch/nn/commit/b339dad44739267bd79a0eae0fb158bfcab5991b needs to be ported to clnn, and then https://github.com/ViliusT/clnn/compare/master...ViliusT:feature/spatialFullConv should build fine(ish)