-
Notifications
You must be signed in to change notification settings - Fork 69
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
Error in convertArray: all(probes1$ProbeSeqA == probes2$ProbeSeqB) is not TRUE #85
Comments
This code is being tested on the test machines. It also works on my
I am thinking that your versions are not in sync., what is the output of
My
On Thu, Nov 10, 2016 at 1:03 PM, ekarlins notifications@github.com wrote:
|
Kasper, Below is sessionInfo on our cluster. Thanks!
locale: attached base packages: other attached packages: loaded via a namespace (and not attached): |
Your annotation packages on your cluster are out of date. Importantly, for Best, On Thu, Nov 10, 2016 at 2:18 PM, ekarlins notifications@github.com wrote:
|
Kasper, To be clear if others read this, the minfi function "convertArray" works well. The issue was I made my RGSet using an old annotation file which caused the function to break. Thank you Kasper for updating minfi so quickly to work with EPIC arrays! It's really impressive!! best, |
When I try to convert an EPIC RGSet to a 450k RGSet using minfi 1.20.0 I get the following error:
It looks like the problem is in lines 303 and 304 of https://github.com/kasperdanielhansen/minfi/blob/master/R/combineArrays.R
The code reads:
stopifnot(all(probes1$ProbeSeqA == probes2$ProbeSeqB))
stopifnot(all(probes1$ProbeSeqB == probes2$ProbeSeqA))
So currently it's:
A == B
B == A
If you change it to:
A == A
B == B
The code runs as expected.
Is this a typo?
Thanks!
Eric
The text was updated successfully, but these errors were encountered: