ang in make_pol_grid is a float and doesn't have reshape method
RichardPflaum opened this issue · comments
RichardPflaum commented
Hi, I just cloned this and am trying to understand the module, but my Linter is complaining:
The line ang = self.angular(k, theta_i).reshape((L, 1), order='F')
in the method make_pol_grid seems to be wrong, since self.angular returns a float.
RichardPflaum commented
Sorry, nevermind this must be a Pylint issue.
RichardPflaum commented
The issue is caused by the angular
method in the baseclass. This can be fixed using ABCMeta
as metaclass
for Zern and using the @abstractmethod
decorator for angular instead of simply returning 0.
Jacopo Antonello commented
Hi. Could you please check whether a1d85c3 fixes this problem? My linter didn't show any warning, so I can't reproduce this. Thank you.
RichardPflaum commented
Looks good! I'll check it tomorrow.
RichardPflaum commented
Yes that does it. Thanks!