-
Notifications
You must be signed in to change notification settings - Fork 48
No IMS model ran mzML input #117
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
Comments
Can you share which converter you used, and the mzML for the full spectrum (or the whole file)? I am going to guess that this has to do with IMS being written to different parts of the XML document by different converters. |
These files were converted by MSFragger. The gzipped mzML can be downloaded via: https://filesender.belnet.be/?s=download&token=0f657e1d-ea0c-4c75-8b89-227cf2d1394c Random example spectrum:
|
ok ... the issue is that I was expecting the 1/k0 info to be inside the selected ion info
whilst in your data it is located in the scan info
... I think according to the current mzML specification both are viable annotations, it is both a:
https://github.com/HUPO-PSI/psi-ms-CV/blob/master/psi-ms.obo line 18481 The solution would be to copy the content here: to this line as well ... IN THE MEANTIME you can pass the .d directly! (if you have one ...) |
(possible) fix there |
Thanks for the help! I can work with this. |
Hey,
I was eager to test the new ims model on some of our data. I use MS2-only mzML files that contain ion mobility fields as:
<cvParam cvRef="MS" accession="MS:1002815" name="inverse reduced ion mobility" value="1.1835660934448242"/>
After succesfully running sage, the PIN and TSV outputs the ion_mobility, predicted_mobility and sqrt(delta_mobility) columns all default to zero for all PSMs. Do we need to specify another JSON parameter or the info might not be correctly parsed from the mzML?
Thanks!
The text was updated successfully, but these errors were encountered: