Issue 1006: ImportTransitionListColumnSelectDlg too trusting of column header names

Assigned To:Brian Pratt
Opened:2024-01-25 16:14 by Brendan MacLean
Changed:2024-01-29 15:45 by Brendan MacLean
Related Issues
2024-01-25 16:14 Brendan MacLean
Title»ImportTransitionListColumnSelectDlg too trusting of column header names
Assigned To»Brian Pratt
Create a simple one peptide document, e.g. type in "PEPTIDER".
View > Document Grid.
Reports > Peptide Transition List.
Click the Copy All button in the Document Grid.
Paste this transition list back into the Skyline document.

1) Skyline shows the ImportTransitionListColumnSelectDlg with the "Collision Energy" column set to "Explicit Collision Energy" despite the fact that all values are zero. Should they be #N/A? This does not cause an error, but if you leave it this way all transitions will export 0 for CE no matter how you change the CE predictor... Didn't you say you wanted a zero CD?
2) Skyline shows the "Library Intensity" column set to "Library Intensity" despite the fact that all entries are #N/A. This causes an error until you manually choose "Ignore Column" for this column.

2024-01-25 16:31 Brendan MacLean
Oops. This is only when Skyline fails to detect the Product m/z column, which I caused by adding reporter ions. If Skyline correctly detects the Product m/z column, then these columns get set to "Ignore Column". Strange, it clearly knows this is a peptide list because it knows the sequence column and the Precursor m/z column, but these two columns getting not ignored means I end up having to change 3 column selections rather than just pointing Skyline to the Product m/z column that it was unable to detect.

2024-01-29 10:06 Brian Pratt
see issue 1002 for related small molecule issues

2024-01-29 15:45 Brendan MacLean
Wrong again. This really is a problem with no apparent solution yet. The time it chose correctly for me was after I had already successfully completed a paste with the two columns set to "Ignore Column" and it was the forms column selection memory that kept it that way for the next paste. If I change the column layout between tries, then it will always incorrectly enabled "Explicit Collision Energy" for a column with all zeros, and "Library Intensity" for a column with only "#N/A".