search

csdms-contrib / slepian_alpha

Scalar spherical-harmonic analysis and Slepian functions
GNU General Public License v2.0
26 stars 31 forks source link

localization issue with nargout #8

Open AlainPlattner opened 8 years ago

AlainPlattner commented 8 years ago

When I call localization with less than 5 output arguments then it does not return anything. This is because of line 71.

I suggest to remove the if-statement on line 71. Is there any reason why it should stay?

harig00 commented 8 years ago

I'm ok with removing it. Ask FJS why its in there. Do you use localization a lot? I never use it so its not a problem I've come up against.

AlainPlattner commented 8 years ago

I had not used it before but it is a function that is in there and Tolu used it (he used it correctly with without problems). But when I tried to use it I stumbled over this issue.

Chris Harig notifications@github.com writes:

I'm ok with removing it. Ask FJS why its in there. Do you use localization a lot? I never use it so its not a problem I've come up against.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.*