Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing import locfit::lp #7

Closed
lcolladotor opened this issue Mar 24, 2016 · 3 comments
Closed

Missing import locfit::lp #7

lcolladotor opened this issue Mar 24, 2016 · 3 comments

Comments

@lcolladotor
Copy link
Contributor

Hi,

Currently this fails in derfinder:

> library('derfinder')
> 
> 
> prep <- preprocessCoverage(genomeData, cutoff=0, scalefac=32, chunksize=1e3, 
+     colsubset=NULL)
> regs1 <- findRegions(prep$position, genomeFstats, 'chr21', verbose=TRUE, smooth = TRUE)
2016-03-24 14:38:18 findRegions: smoothing
Error in eval(expr, envir, enclos) : could not find function "lp"

> packageVersion('derfinder')
[1] ‘1.5.29’
> packageVersion('bumphunter')
[1] ‘1.11.3’
> R.version.string
[1] "R version 3.3.0 alpha (2016-03-23 r70368)"

Note that the following works as expected:

> regs2 <- findRegions(prep$position, genomeFstats, 'chr21', verbose=TRUE, smooth = TRUE, smoothFunction = bumphunter::runmedByCluster)
2016-03-24 14:38:18 findRegions: smoothing
2016-03-24 14:38:19 findRegions: identifying potential segments
2016-03-24 14:38:19 findRegions: segmenting information
2016-03-24 14:38:19 .getSegmentsRle: segmenting with cutoff(s) 19.7936614060235
2016-03-24 14:38:19 findRegions: identifying candidate regions
2016-03-24 14:38:19 findRegions: identifying region clusters

I noticed that you do use the locfit::lp function at https://github.com/ririzarr/bumphunter/blob/master/R/smooth.R#L81 but it's currently not in the NAMESPACE file.

Best,
Leo

@lcolladotor
Copy link
Contributor Author

Sent a SVN patch as reported in the bioc-devel mailing list due to #9

@martinaryee
Copy link
Collaborator

Hi Leo,

After some issues setting up the git/svn mirror your patch is now merged!

@lcolladotor
Copy link
Contributor Author

Awesome! Thanks Martin

sampoll pushed a commit that referenced this issue Oct 4, 2017
From: lcolladotor <lcollado@jhsph.edu>

git-svn-id: file:///home/git/hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/bumphunter@115319 bc3139a8-67e5-0310-9ffc-ced21a209358
sampoll pushed a commit that referenced this issue Nov 2, 2018
From: lcolladotor <lcollado@jhsph.edu>

git-svn-id: file:///home/git/hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/bumphunter@115319 bc3139a8-67e5-0310-9ffc-ced21a209358
sampoll pushed a commit that referenced this issue Nov 5, 2018
From: lcolladotor <lcollado@jhsph.edu>

git-svn-id: file:///home/git/hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/bumphunter@115319 bc3139a8-67e5-0310-9ffc-ced21a209358
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants