mzLib icon indicating copy to clipboard operation
mzLib copied to clipboard

support for <referenceableParamGroup> / SCIEX WIFF converter

Open ftwkoopmans opened this issue 5 years ago • 2 comments

WIFF files converted to mzML using the SCIEX data converter yield an error in MetaMorpheus, Run failed, Exception: !msOrder.HasValue || !isCentroid.HasValue, while converting the same WIFF using ProteoWizard's qtofpeakpicker works great.

A key difference is probably the extensive use of <referenceableParamGroupRef> in the mzML files generated by the SCIEX tool, perhaps the enclosed properties are not taken into account when parsing a <spectrum> element. We worked through a similar problem with pymzML a while ago, please check the mzML snippet I posted at @ https://github.com/pymzml/pymzML/issues/92

Looking through the test mzML files in the mzLib repository, I found one that does contain a <referenceableParamGroupRef> (https://github.com/smith-chem-wisc/mzLib/blob/84a58e9cd62feb7bf38cc928c6a7b0b132b18a9b/Test/tiny.pwiz.1.1.mzML) element but it seems that the required <cvParam cvRef="MS" accession="MS:1000127" name="centroid spectrum" value=""/> is not in the <referenceableParamGroupRef> as it is in my mzML files but instead is repeated for every <spectrum>. So perhaps a few minor adjustments (eg; moving all "centroid spectrum" and "ms level" properties to the param group) to that test mzML could create a mock mzML analogous to my use-case so you can unit test this in the future.

I've uploaded an mzML generated by both qtofpeakpicker and the SCIEX tool for testing @ https://surfdrive.surf.nl/files/index.php/s/pV1GXjfPHONyajx

ftwkoopmans avatar Mar 20 '19 19:03 ftwkoopmans