-
Notifications
You must be signed in to change notification settings - Fork 80
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
adjustRtime not conducted with large dataset, but working after register(SerialParam()) #766
Comments
Thanks for reporting. Could you maybe also specify the method and parameters you used for the alignment? Also, did you use the newer objects that use/base on |
Hi @jorainer Many thanks for your reply, and my apologies for my late response - as described below I succeeded with the SerialParam workaround, and now completed re-processing to look into the data. I have the script running for a while, so likely not use the latest, but it has been working for several previous dataset expansions re-running the analysis (until the latest). Here are the data preprocessing steps I used, data input is .mzXML (single polarity, positive mode, converted via ProteoWizard):
And as the last did not go through, I instead on Ubuntu 24.04.1 LTS with R version 4.4.1 (2024-06-14) and xcms_4.2.3 ran the following, which worked. This workaround did not work on Windows10 though:
From there, I succeeded to complete the processing, and can't see any further problem in the output. |
Just for curiosity - what are the dimensions of your To fix that, we're working on a solution that uses also an on-disk representation of the |
Thanks, sounds good! Regarding your enquiries, I checked as outlined the dimensions of xdata of one of the datasets, at the stage before failing the retention time alignment (i.e. steps conducted were data reading and peak picking only):
Many thanks! |
This is a bit puzzling. The data is actually quite small, so I can not really understand why parallel processing should not work. What type of parallel processing are you using? |
Hi jorainer I do not enable parallel processing beforehand in a separate command. So at the stage of retention time alignment, the XCMS processing functions are really just run as noted above (copied below for reference):
And as noted, serialisation permitted to complete the retention time alignment, but only on an Ubuntu computer and not on my Windows10 computer
Many thanks and best wishes Christian |
With a large dataset of >500 injections (Orbitrap Q-Exactive, full scan, not centroided), I encountered that the retention time adjustment did not execute. The script kept running, so that later processing steps continued though no adjusted retention time was reported.
As workaround, the adjustRtime function worked fine when I ran beforehand register(SerialParam()), as it was proposed for a similar issue here: #358
For reference, the data used is mostly in the Metabolights-repository under MTBLS3450 and MTBLS8433 (for analysis, only the positive-mode acquisitions were used, while measurement was with alternating polarity). The latest extension of the dataset, with which the error occured, is not yet in the repository. For XCMS, data was converted to mzXML via Proteowizard.
The dataset comprises time-series (sampling) data from different groundwater probing sites ("Location" in metabolights), and one location (H14, smallest number of injections) executed in XCMS without problems, while 7 locations (with >500 injections) showed the error.
I observed the issue both on Windows10, and Ubuntu. The workaround was successfully tested on Ubuntu (only for now), with the following sessionInfo:
The text was updated successfully, but these errors were encountered: