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

MS5 and "merged MSn" spectra #403

Closed
meowcat opened this issue Apr 5, 2024 · 4 comments
Closed

MS5 and "merged MSn" spectra #403

meowcat opened this issue Apr 5, 2024 · 4 comments

Comments

@meowcat
Copy link
Contributor

meowcat commented Apr 5, 2024

Hi,

I am working with a contributor who collected a ton of MSn spectra. Two things:

  • they have MS5 spectra, which are not currently in RecordParserDefinition.java. Can we add MS5 to the definition?
  • they also have "merged MSn" spectra, where all MSn spectra have been pooled to a single spectrum. I would suggest to simply add "MSn" as a definition for those. Would you agree? Or do we want to avoid these?
@schymane
Copy link
Member

schymane commented Apr 6, 2024

I can't help with point (1) but agree that it should be added. Regarding point (2), I think these spectra should be included (not avoided), either as "MSn" or "merged MSn"?

@sneumann
Copy link
Member

sneumann commented Apr 8, 2024

+1 for having the spectra. MS5 can be added right away.

Merged MSn is interesting, and we definitely want those..
It is similar to the idea of ramped spectra, which also are a "merge" of multiple collision energies.
We could add 'em as AC$MASS_SPECTROMETRY: MS_TYPE MSn, and the merging would be mentioned in the MS$DATA_PROCESSING: merged MSn
What about the focused ion, maybe only the MS2 ? What if they also have in-source fragments in another MS2 scan ? Can there be multiple collision energies between levels ? Maybe the title mentions "Merged MSn" ?
Yours,
Steffen

@meowcat
Copy link
Contributor Author

meowcat commented Apr 8, 2024

For the FOCUSED_ION we could either
PRECURSOR_M/Z 581.18649/merged
or just like you suggest
PRECURSOR_M/Z 581.18649

The slashes exist anyway in MSn records so the field is already not numeric. (Same, actually, for the collision energies, both for MSn and for all the stepped/merged MS2.) From that point of view, "merged" would not cause much more damage... One could argue that it's not extra information if the record says "MSn merged" anyway. I guess your suggestion is better.

The suggestion for DATA_PROCESSING would currently be
MS$DATA_PROCESSING: SPECTYPE ALL_MSN_TO_PSEUDO_MS2

@meier-rene
Copy link
Contributor

This is included in release 2.2.6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants