-
Notifications
You must be signed in to change notification settings - Fork 70
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
readmetharray force=TRUE #92
Comments
You should only hit line 58 if you're trying to read data from different arrays (which is not supported and I don't intend to support it). The situation with data from scans with different lengths of the same array is different; thats what we support. Perhaps tell us more details or give two files as example. |
The file sizes are 13677911 instead of 13686979 and the number of probes are 1051943. Setting the annotation manually to "ilm10b2.hg19” works fine, however around 600 CpG probes are missing. The same probes seem to miss when reading the idat files with GenomeStudio. |
Fixed in minfi 1.21.6. It will take a few days before it appears on the Bioconductor servers. |
Thanks, I think the problematic probes are excluded by the new Illumina manifest released today. |
No, we don't use the manifest when reading the files. But the problem
should be solved in the devel version of minfi for example version 1.21.6
or higher.
…On Wed, Apr 19, 2017 at 1:40 PM, Martin Sill ***@***.***> wrote:
Thanks, I think the problematic probes are excluded by the new Illumina
manifest released today.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#92 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEuhnygrMYj24CAdnJhkQ0JMkPtgQaqGks5rxkb5gaJpZM4LtXBP>
.
|
I'd like to ask why would scans of different lengths happen? As it seems to be a constant case if EPIC's arrays [+Isn't it supposed to be all done uniformly by a machine? ]
|
Hi Kasper,
thanks for the hard work on the package. I am trying to read epic idats of different file sizes and I think you need to uncomment/delete line 58 in in read.meth.R to get the force=TRUE argument working?
Best,
Martin
The text was updated successfully, but these errors were encountered: