- changed status to resolved
Remove subsample argument from findThreshold?
Issue #113
resolved
With the new subsample
argument to distToNearest
, and if stick with the new density threshold bandwidth detection, we probably don't need the subample
argument in findThreshold
.
It's also inconsistent because it doesn't apply to method="gmm"
.
Comments (1)
-
reporter - Log in to comment
Moved
subsample
out ofsmoothValley
and intofindThreshold
before either method. Also setsubsample=NULL
as it no longer seems necessary.