allow the user to override the default smoothing parameter for sources

[baggepinnen]

No description provided.

[codecov]

Codecov Report

Merging #146 (13d8827) into main (a99c4cc) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #146      +/-   ##
==========================================
- Coverage   67.51%   67.48%   -0.03%     
==========================================
  Files          33       33              
  Lines        1499     1498       -1     
==========================================
- Hits         1012     1011       -1     
  Misses        487      487              

Impacted Files	Coverage Δ
src/Blocks/sources.jl	92.38% <100.00%> (-0.08%)	⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[ven-k]

Changes LGTM.

But wouldn't it be better if

1. We have dispatches based on smooth - Int or Bool?
   Or
2. Just have another var δ or `smooth_coef``

[baggepinnen]

1. You cannot dispatch on keyword arguments, and the workaround to do so makes the code very bloated
2. I didn't want two different arguments to handle the same thing, both of which must be set to achieve a change in smoothness factor
3. I don't want to break the existing interface.

I think this solution was a good tradeoff

[jebej]

Doesn’t this change make those functions type unstable when smooth::Bool?

[baggepinnen]

They do, but the output is a symbolical expression, which is type unstable anyways so what does it matter?