lazear / sage

Proteomics search & quantification so fast that it feels like magic
https://sage-docs.vercel.app
MIT License
201 stars 38 forks source link

Fix to #117 #119

Closed jspaezp closed 4 months ago

jspaezp commented 5 months ago

Adds the option to extract ion mobilities from mzML files that encode it in the scan section instead of the selected ion section (which was not something i knew could happen)

https://github.com/lazear/sage/issues/117

lazear commented 4 months ago

I would add a async fn parse_spectrum_issue_117() with the offending spectrum in question (removing some non-essential fields is OK too) to make sure it parses

jspaezp commented 4 months ago

Added the test ... there isn't really that many non-essential fields that can be removed. but there it goes :)