Issue 921: Transition list reader should identify special ions by m/z the same way it does calculated peptide fragment ions

issues
Status:open
Assigned To:Brendan MacLean
Type:Todo
Area:Skyline
Priority:3
Milestone:4.3
Opened:2022-11-07 11:05 by Brian Pratt
Changed:2024-01-22 11:35 by Brian Pratt
Resolved:
Resolution:
Related:1000
Closed:
Related Issues
2022-11-07 11:05 Brian Pratt
Title»Transition list reader should identify special ions by m/z the same way it does calculated peptide fragment ions
Assigned To»Brian Pratt
Notify»philip remes
Type»Todo
Area»Skyline
Priority»3
From an email thread 11/7/2022 with Nick, Phil Remes, Brendan, BrianP

Brendan,

It looks like "File > Import > Transition List" doesn't know how to deal with the "Special Ions" that might be listed on "Settings > Transition Settings > Filter".
Philip Remes has a huge transition list with many TMT and non-TMT transitions in it.
Is there any way that he could get all of those transitions into the document?

One thing that I thought would work would be first to have a document with all of the TMT transitions in it, and then use "File > Import > Transition List" to import a CSV file containing all of the non-TMT transitions in it, and then he would end up with peptides with the complete set of transitions that he wants. Unfortunately, doing that always results in a second peptide group being created, so that there is no peptide in the document which contains the complete set of transitions that it is supposed to.

Can you think of a way for Philip to get all of the transitions into the document?

I'm thinking we should just fix it so that MassListRowReader knows what to do when it sees a transition whose m/z matches a Special Ion.
-- Nick
 
 yeast11plex_ms2_subset0_transitions.csv
 221028_yeast_60min_tms2_0.sky.zip-rename

2022-11-07 14:59 Brian Pratt
Nick adds:

I wonder if we should combine fixing this bug with this other bug that's currently assigned to Rita:
Issue 352: Annotate reporter ions in Library Match graph

There are at least three different places in the product where we call "SequenceCalc.GetFragmentIonMasses".
Those places are:
TransitionGroup.GetTransitions: populating the list of transitions that could be added to a TransitionGroup
MassListRowReader.CalcTransitionExplanations: importing a transition list
SpectrumRanker constructor: Annotating ions in a library spectrum

In each of those places we've either already added or will have to add in the future code to deal with neutral losses, reporter ions, and crosslinked peptides.
I am pretty sure that each of those things should know about exactly the same types of ions, and any differences in the implementation in those areas will lead to bugs.

I imagine we should probably centralize that logic somewhere so that no one calls "GetFragmentIonMasses" directly.
-- Nick

2024-01-22 09:52 Brian Pratt
Related»1000

2024-01-22 11:35 Brian Pratt
Assigned ToBrian Pratt»Brendan MacLean