You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
will fail, because the number of filters is 16 which doesn't pass the assert check. However, the documentation here mentions that this number should be a multiple of 16.
(1) The comment right by the check says there's a bug in the calling code blow, i haven't really looked into it (the comment is from Alex)
(2) I am not sure
I think you can try removing the assert (change the assert to %16) and run 1000 iterations of the unit tests with batch-size as a multiple of 16, and that might uncover any issues. The unit tests are pretty robust and will uncover any issues.
The following assert exists in cuda-convent2:
https://github.com/soumith/cuda-convnet2.torch/blob/master/cudaconv3/src/img_acts.cu#L1208
This causes failure in some cases that cuda-convnet2 should support, for example:
will fail, because the number of filters is 16 which doesn't pass the assert check. However, the documentation here mentions that this number should be a multiple of 16.
https://github.com/soumith/cuda-convnet2.torch/blob/master/cudaconv3/src/img_acts.cu#L1345
does this mean that removing the assert will make the code above work correctly?
The text was updated successfully, but these errors were encountered: