When you previously submitted this PR, I asked you why the base class had to be specific to convolution. I don't see that addressed here.