-
Notifications
You must be signed in to change notification settings - Fork 2
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
Manhatten plot - multiple charge states #8
Comments
Hi Nathan, yes this will be a charge related issue. Could you send me a small example and then I can fix the function? |
Yes, I'll put something in our fileshare today. |
this should be fixed in the lastest issue. If you just pick one charge state everything should be fine, need to work out what we visualise when there are multiple charge states. |
Thanks for working on that Olly; I'm till getting an issue in which R is removing data for 91 peptides (which is all of them) in the Manhattan plot. The diffdata, region and sequences objects all contain data. I don't know if this arning message contains any immediate ideas:- "Warning message: |
I fixed the issue now; although no get a problem where there are more peptides (plotted on the x-axis) than p-values; hdxstats has not left out a gap / removed peptides with no data, and so now there ar a bunch of peptides at the end with no data, and presumably the other p-values are now misaligned |
Hard to see how this happens without some data, anything you could share? |
I put an example in the fileshare that should be more helpful. |
manhattan/multiple charge states reminder |
Generating a Manhatten plot with some alternative data, I encounter this issue:-
Error in
$<-.data.frame(
tmp, "protection", value = c(XXXXXXX_2 = 1, : replacement has 96 rows, data has 91
So from what I can gather, the issue is that the original data frame has 96 unique peptides, but the HdxStatsRes object, the function only sees 91. This is because in HdxStatsRes, the charge state is concatenated in; however, region = protein[, c("Start", "End")] does not take into account that you might have several peptides in the dataset with the same ID but different charge state. Is it as easy a fix as simply adding in "Charge" to that line of code? I haven't managed to get the Manhatten plot with my data, so haven't managed to test that out. (I'm getting some errors with that I haven't been able to resolve yet)
The text was updated successfully, but these errors were encountered: