Change defintion of padding in "MyConv2d" about lernapparat HOT 3 OPEN

I'm not sure what this would change?
w.size(-1) is an int and so is w.size(-1)-1.

from lernapparat.

Ah, now I see, when tracing, the .size returns a tensor.
I must admit I'm not exactly thrilled to hack around it with int. It would be much cleaner to wrap the call into a script function if you need tracing.
That would work better in general, too, as it doesn't rely on the fixed weight size (which, I admit, is not critical if you want to trace the model here).

from lernapparat.

Thanks for your reply! I am trying to export the model as .onnx and I made the change to get rid of an error message saying that padding should be a tuple of ints and not a tensor. In your opinion, did I mess anything up with such a change?

from lernapparat.