Remove subsample argument from findThreshold?

Issue #113 resolved
Jason Vander Heiden created an issue

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)

  1. Jason Vander Heiden reporter

    Moved subsample out of smoothValley and into findThreshold before either method. Also set subsample=NULL as it no longer seems necessary.

  2. Log in to comment