edgeR minor bug in normalizeChIPtoInput
2
0
Entering edit mode
dalerr • 0
@dalerr-12273
Last seen 7.8 years ago

Very minor issue; I noticed that some return calls in normalizeChIPtoInput.R are missing list() at

https://github.com/Bioconductor-mirror/edgeR/blob/master/R/normalizeChIPtoInput.R#L24

which will give an error

Error in return(p = rep(0, 1), scaling.factor = 0, prop.enriched = 1) : multi-argument returns are not permitted.

It's not clear how/where to submit patches, but the github repo says bug reports should be submitted here. Sorry for the noise if this is not the right venue.

edgeR • 957 views
ADD COMMENT
0
Entering edit mode
Aaron Lun ★ 28k
@alun
Last seen 3 hours ago
The city by the bay

Thanks for spotting this. Yes, this is a bug, and yes, this is the right place to report it.

ADD COMMENT
0
Entering edit mode
@gordon-smyth
Last seen 2 hours ago
WEHI, Melbourne, Australia

Posting a bug here is ok, but most Bioconductor package authors prefer for bug reports to be mailed directly to them, to the email address given in the Maintainer field of the DESCRIPTION file. Section 1.3 of the edgeR User's Guide says:

"The edgeR authors always appreciate receiving reports of bugs in the package functions or
in the documentation. The same goes for well-considered suggestions for improvements. All
other questions or problems concerning edgeR should be posted to the Bioconductor support site."

Anyway, thanks for pointing it out.

ADD COMMENT

Login before adding your answer.

Traffic: 610 users visited in the last hour
Help About
FAQ
Access RSS
API
Stats

Use of this site constitutes acceptance of our User Agreement and Privacy Policy.

Powered by the version 2.3.6