hansenlab / minfi

Devel repository for minfi
58 stars 67 forks source link

RatioSet() documentation unclear / or issue with annotation name parsing #133

Open RichardJActon opened 6 years ago

RichardJActon commented 6 years ago

the annotation= option to RatioSet() is documented as "An annotation string, optional."

however, it appears to require an argument of the form annotation = c(array = "IlluminaHumanMethylation450k",annotation = "ilmn12.hg19")

just giving annotation = "IlluminaHumanMethylation450k.ilmn12.hg19" throws an error.

Loading required package IlluminaHumanMethylation450k.ilmn12.hg19anno
there is no package called 'IlluminaHumanMethylation450k.ilmn12.hg19anno'Error in getAnnotationObject(object) :
...

This could be fixed by making .getAnnotationString() in utils.R able to parse the whole string or at least by updating the documentation to indicate the form in which this argument is expected.

(took me a while to figure this out - thought I had some kind of version issue)