-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Closed
Labels
bugsomething brokensomething broken
Description
Regarding this line of code in plotly.js:
plotly.js/src/traces/violin/calc.js
Line 51 in 9d61443
var n = Math.ceil(dist / (Math.min(bandwidthDflt, bandwidth) / 3)); |
Why is taking the minimum (Math.min(bandwidthDflt, bandwidth)
) needed here? Should it just not take the min and use only bandwidth
? I can see a problem where I defined trace.bandwidth
and don't want to use bandwidthDflt
.
Metadata
Metadata
Assignees
Labels
bugsomething brokensomething broken