enh: added option to smooth LDOS grid with Gaussian smoothing from sisl

[sofiasanz]

Added possibility to smooth the real space grid with Gaussian smoothing for LDOS plots.

[zerothi]

Just an idea, you could let smooth be a dict which contains the arguments for the function, then you don't need to think too much about additional parameters?

[sofiasanz]

Just an idea, you could let smooth be a dict which contains the arguments for the function, then you don't need to think too much about additional parameters?

Yes I like it! You can see my implementation in commit cf3d5ec.

[sofiasanz]

I see! Yes that is much better :-)

[zerothi]

I don't know if this what you want, but this will always smooth it?

[sofiasanz]

But if the user doesn't pass any arguments for smooth then it assumes r=0, no? And then it doesn't have the smoothing effect, no?

[sofiasanz]

Otherwise it can just first check if the smooth dictionary is empty or not.

[zerothi]

But if the user doesn't pass any arguments for smooth then it assumes r=0, no? And then it doesn't have the smoothing effect, no?

No, it will use the default values, https://zerothi.github.io/sisl/api/generated/sisl.Grid.html?highlight=smooth#sisl.Grid.smooth

Remember that kwargs for a zero length dict is effectively passing no arguments. I think it is safest to check whether the argument exist (checking the length would not allow users to use default values).

[sofiasanz]

I see. Forgot to check which were the default values of the method. Thanks!

Remember that kwargs for a zero length dict is effectively passing no arguments. I think it is safest to check whether the argument exist (checking the length would not allow users to use default values).

I just saw again what you wrote in #123 (comment), didn't see the first line before hehe

[zerothi]

Lgtm!