You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Remember that array-blur evolved from this module, but became faster (about 50% if I remember correctly), more versatile, well-tested etc.
I can incorporate it back into d3-contour (in src option A, or as a dependency option B), but my preference would still be option C, to make it part of d3-array (d3/d3-array#151, which I would have to update). It's true that the blur() method has a provision for a "width" that allows to grant a pixel matrix topology to the array, but width defaults to 1 and it's a useful 1-d transform, for example for time series.
blurring could also be a useful channel transform for Plot (as a complement to windowX, with pros and cons).
Let me know which route you'd prefer. Happy to do the change in any case.
d3.contourDensity internally rounds the bandwidth:
d3-contour/src/density.js
Line 153 in 06f2cf1
It would be nice to avoid rounding here so that arbitrary bandwidths could be used. Maybe this is possible if we switch to using array-blur, @Fil?
The text was updated successfully, but these errors were encountered: