xcms icon indicating copy to clipboard operation
xcms copied to clipboard

status of Warpgroup

Open tentrillion opened this issue 5 years ago • 4 comments

XCMS team, I was recently re-reading the Warpgroup paper from Gary Patti's group. I think this algorithm came out before xcms3. I was wondering if any of these ideas would make sense to integrate into XCMS.

Does xcms3 make the Warpgroup technique superfluous? Looking forward to thoughts on the issue. I'd also be happy to help if it is useful. I'm a pretty experienced R user and modestly experienced developer, for a wet lab person.

Potential labels for this issue: xcms3, enhancement

tentrillion avatar Mar 21 '19 16:03 tentrillion

We haven't integrated warpgroup into xcms. To me that sounds like an interesting idea and contributions in the form of pull requests are anyway highly welcome! (@sneumann any comments?).

jorainer avatar Mar 25 '19 08:03 jorainer

I am working with samples that contain stable-isotope labels and I find that WarpGroup handles the grouping and detection of peaks much better than CentWave and Obiwarp. The main issue I run into is that WarpGroup (and X13CMS which I also utilize) rely on xcmsSet objects while xcms3 now creates XCMSnExp objects. I am able to convert my XCMSnExp objects down to xcmsSet objects but it comes at the cost of losing my MS2 data. Is there a way around this issue?

bouranij avatar Oct 31 '19 17:10 bouranij

Hi @bouranij, I am afraid not, which was the reason we introduced XCMSnExp :-) maybe you could keep some mapping information between xcmsSet and XCMSnExp ? I could imagine that the authors of those package would appreciate help porting their packages to xcms3. Depending on the packages, porting might be fairly straightforward. Yours, Steffen

sneumann avatar Oct 31 '19 19:10 sneumann

Hi @tentrillion , the abstract of the Patti paper says The package includes example data and XCMS compatibility wrappers for ease of use adn as said above, I'd guess @nathaniel-mahieu wouldn't mind a PR adding a wrapper that takes an xcms3 object. Yours, Steffen

sneumann avatar Oct 31 '19 19:10 sneumann