QUVA-Lab / e2cnn

E(2)-Equivariant CNNs Library for Pytorch
https://quva-lab.github.io/e2cnn/
Other
596 stars 75 forks source link

Add missing discretization argument #54

Closed ejnnr closed 2 years ago

ejnnr commented 2 years ago

PDOs currently seem broken because the discretization argument isn't passed on in one place. From a brief look, I'd guess that this was introduced in https://github.com/QUVA-Lab/e2cnn/commit/135f964be8793171ef1e1f7cdf7fe0bf5990a0c4 (since discretization was part of **kwargs before then). Should be fixed with this PR.

Gabri95 commented 2 years ago

hi @ejnnr

thanks for spotting this bug and for the pull-request!

Gabriele