-
Notifications
You must be signed in to change notification settings - Fork 47
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
Exception during conversion #126
Comments
It seems ThermoRawFileParser produces partial mzML output with 239 scans. The original file has 1280 scans. The conversion crashes at scan 240, which is the first MS level 4 scan... so I suspect it has something to do with the MS level. |
Hi Tomas, |
It has been a long time indeed. I have implemented general parsing for MSn scan, i.e. now it should support all possible MS levels. The branch with the feature is 7b7854d and it will be included in the next release. @tomas-pluskal would you like to test it and provide some feedback? and if so let me know if you build from the source or I should build a beta version for you. |
Hi @caetera |
Hi @tomas-pluskal, Here is the latest beta build: https://syddanskuni-my.sharepoint.com/:u:/g/personal/vgor_bmb_sdu_dk/EeJ50TcZl35OpplVv2ZT7hEBL4gudhEsGoFz1UHgOJXzLw?e=cyvvaS |
Hi @caetera, I tried the converter with our Orbitrap-IDX data (up to MS5) and it worked well. Attached you can find an MS4 scan and all precursors are marked with a dashed line. Thanks for fixing the issue. |
Hi @corinnabrungs, |
@caetera Could you please make a new release now that this bug has been fixed? |
New release has been published. |
Thanks a lot! :) |
Hi, I downloaded the latest ThermoRawFileParser 1.3.4 and I see this exception when parsing a .raw file:
The raw file is attached.
202112_DIMS_MS4_DDA_Orbitrap_IDX.zip
(originally reported by @robinschmid)
The text was updated successfully, but these errors were encountered: