Hello Bioconductor community,
In a continued effort to better code, there have been efforts to identify
problematic existing code in packages. The current issue at hand is identifying
a conditional with a length greater than 1. This could either be an over site
of not using any() or all() or could potential indicate problematic code with
unexpected multiple output.
dummy example:
```
> check = c(1:5)
> if (check > 3){
+ FALSE
+ }
Warning message:
In if (check > 3) { :
the condition has length > 1 and only the first element will be used
```
The following are packages that currently have code with this type of
Warning/ERROR:
cbaf
COHCAP
exomePeak
ExtDist
fabia
gaia
gCrisprTools
GenVisR
Guitar
hapFabia
KEGGlincs
MassArray
miRBaseVersions.db
NOISeq
pathview
rHVDM
SEPA
sights
support
SVAPLSseq
Please correct these issues as soon as possible. For a link to detailed output
on what code is causing this, please see
https://github.com/kalibera/rifcond
We are implementing the check on incoming packages submitted to the
Bioconductor tracker that will produce an ERROR and should be active by the end
of next week.
Thank you.
Lori Shepherd
Bioconductor Core Team
Roswell Park Cancer Institute
Department of Biostatistics & Bioinformatics
Elm & Carlton Streets
Buffalo, New York 14263
This email message may contain legally privileged and/or confidential
information. If you are not the intended recipient(s), or the employee or
agent responsible for the delivery of this message to the intended
recipient(s), you are hereby notified that any disclosure, copying,
distribution, or use of this email message is prohibited. If you have received
this message in error, please notify the sender immediately by e-mail and
delete this email message from your computer. Thank you.
[[alternative HTML version deleted]]
_______________________________________________
Bioc-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/bioc-devel
