-
Notifications
You must be signed in to change notification settings - Fork 7
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
New format of modifications from MaxQuant cause artmsProtein2SiteConversion to fail #166
Comments
Thanks @bpolacco, indeed this is a pretty important issue. Let me get back to this very soon. |
I tested and your code works well. Thank you very much. But I still cannot understand this bizarre change in MaxQuant. I am afraid this is a bug: just a wrong mapping. I bet they will correct it. |
+ Convert new MaxQuant format of PTMs to the old format + MSstats messages are not displayed by default when using artmsQuantification. The user can enable MSstats messages by selecting "display_msstats = TRUE" + Prevent artmsWriteConfigYamlFile() from overwriting an existing configuration file unless the user allows it (overwrite = TRUE) + printPDF now available in all functions printing plots to pdf, which means that notebooks can be used and print all plots. Default is still printPDF = TRUE + Fix bugs #166 #165
Hi @bpolacco i would like to confirm that this pre-conversion also takes into consideration that it seems as though the new [Phospho (STY)] format recognises the phosphorylated amino acid is on the right side of the label [Phospho (STY)]? I had not used the previous MaxQuant ph iteration which i understand recognised phosphorylation on the left hand side amino acid? |
I have not seen the case where the modified STY is to the right of the (Phospho (STY)). I may have confused with my example above |
I've been seeing new format for modifications from recent versions of MaxQuant that cause a failure of
artmsProtein2SiteConversion
. Instead of the shortK(ph)
, MaxQuant is now usingS(Phospho (STY))
. I've been pre-converting these MaxQuant files using the function below. Note that this started as a converter for Spectronaut output which uses the similarS[Phospho (STY)]
format--that's the reason for the[[(]
character classes in the regular expression and the variable namespecFormats
andspecModSequence
.I'm happy to put something like this in artMS as a pull request, but inserting this into artMS code will require a bit more restructuring of your code than I am comfortable doing without discussion on how you like to structure things. I'm thinking something like a checkpoint in artmsProtein2SiteConversion, and an attempt to convert on failure, followed by another checkpoint...
evidence_1000lines.txt
The text was updated successfully, but these errors were encountered: