-
Notifications
You must be signed in to change notification settings - Fork 22
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
Problem with LDsearch command #42
Comments
@Sandyyy123 could you post a minimal example that illustrates the problem? |
a <- c("rs11776767") |
Another issue that I had encountered is that if you increase the number of SNPs (>20) in the vector a supplied to the LDSearch command, it fails to give the results |
The problem seems to come from upstream, i.e. the SNAP service itself. Particularly if you're asking for the column "Associated Gene Annotations from GeneCruiser" for the output, i.e. option "GA" for the columnList variable. Thus, this call:
results in error:
So the problem should be reported to SNAP (http://archive.broadinstitute.org/mpg/snap/ldsearch.php). You should be able to reproduce this error from the web interface, just make sure "Associated Gene Annotations from GeneCruiser" option is checked down there in the Output Columns. Nevertheless, this package should probably better handle these errors before they surface down the line... |
@Sandyyy123 Regarding that other issue you mentioned about the number of SNPs supplied, it might be better to submit a separate issue, and again, supply a minimal example that shows how exactly it fails. |
@Sandyyy123 this is likely due to a URL that is too long, but do open an issue for that. tried now, but their server is down - will look at again soon |
Thanks, I am waiting for their server to become functional before I could
post the issue.
On 27 Feb 2017 22:31, "Scott Chamberlain" <notifications@github.com> wrote:
Another issue that I had encountered is that if you increase the number of
SNPs (>20) in the vector a supplied to the LDSearch command, it fails to
give the results
@Sandyyy123 <https://github.com/Sandyyy123> this is likely due to a URL
that is too long, but do open an issue for that.
tried now, but their server is down - will look at again soon
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#42 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/APr4uVfnIuT6K5PbNZSamkkA1ZGlZPEYks5rg0CzgaJpZM4MMDGE>
.
|
@Sandyyy123 @sckott It seems this issue has been fixed in SNAP. |
Thank you. The issue has now been resolved at the SNAP server. |
great, glad to hear |
The LD search command was working perfectly until yesterday. I am having the following issue now:
Error in split_to_df(tmp, sep = "\t", fixed = TRUE) :
non-equal lengths for each entry of x post-splitting
In addition: Warning message:
In if (grepl("NCBI", deparse(sys.call()))) { :
the condition has length > 1 and only the first element will be used
The text was updated successfully, but these errors were encountered: