nn_decomposer does not support kernel_h and kernel_w about caffe HOT 3 OPEN

@hahne There is no constraint on the kernel size from the algorithm perspective. It works in general (see here).

The code expects those, only because I did a lazy job to escape from a cumbersome code block of if-else-if-else to check that the kernel_size exists in kernel_size or kernel_h kernel_w . If it exists in kernel_size, we also had to check it was a list or a scalar to make it work in general.

from caffe.

@wenwei202 ok understand. Thank you for the info!

from caffe.

@hahne I suspect those assert is not necessary for linear combination layer, but we need to delete kernel_h and kernel_w if they are copied from decomposed conv layer. Similar thing for stride, pad and dilation.

from caffe.