Skip to content
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

Hydroxy proline diagnostic ion error #2324

Merged

Conversation

trishorts
Copy link
Contributor

@trishorts trishorts commented Dec 11, 2023

Ben Orsburn in his proteomics news blogpost highlighted an error in the reported mass for the diagnostic ion of hydroxyproline. Turns out we had the erroneous mass in our mods list. This PR updates the mass of the diagnostic ion to the correct value of 171.0764 which comes from C7 H11 N2 O3+ as reported by Dr. Frey.

https://proteomicsnews.blogspot.com/2023/12/hypothesis-shot-down-by-one-of-greatest.html

https://www.ncbi.nlm.nih.gov/pmc/articles/PMC6126394/

@@ -114,7 +114,7 @@ TG P
PP Anywhere.
MT Common Biological
CF O1
DI HCD:170.060123533
DI HCD:170.07642
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the comment, you said that the mass was updated to 176.0764. not 170.07642.

In the blog post, Ben Orsburn said that he pulled an XIC for 176.076, but then he shows a spectra where an ion at 171.0766 is highlighted.

Are the diagnostic ions recorded as masses and then converted to mz downstream? I'm a little confused as to what's going on.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wrong mass in comment. also correct now. forgot to push yesterday before i left. these are mzs so no conversion.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

@trishorts trishorts merged commit 984505d into smith-chem-wisc:master Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants