Hello! I was hoping we could discuss why the lookup of a modification is using the 'NT' name when an unimod accession is available. It sincerely feels like the human readable name should be a fallback or at least checked for consistency when validating the input SDRF.
LMK if we could improve the documentation on the pipeline to know what fields/sub-fields actually matter for the execution.
Error executing process > 'NFCORE_QUANTMS:QUANTMS:DIA:DIANNCONVERT (input_files.sdrf)'
Caused by:
Essential container in task exited
Command executed:
diann_convert.py convert \
--folder ./ \
--exp_design input_files.sdrf_openms_design.tsv \
--diann_version ./version/versions.yml \
--dia_params "0.02;Da;10;ppm;Trypsin;Carbamidomethylation (C);Oxidation (M)" \
--charge 4 \
--missed_cleavages 2 \
--qvalue_threshold 0.01 \
2>&1 | tee convert_report.log
cat <<-END_VERSIONS > versions.yml
"NFCORE_QUANTMS:QUANTMS:DIA:DIANNCONVERT":
pyopenms: $(pip show pyopenms | grep "Version" | awk -F ': ' '{print $2}')
END_VERSIONS
Command exit status:
1
line 422, in MTD_mod_info
mod_obj = mods_db.getModification(mod)
File "pyopenms/pyopenms_2.pyx", line 9057, in pyopenms.pyopenms_2.ModificationsDB.getModification
File "pyopenms/pyopenms_2.pyx", line 9016, in pyopenms.pyopenms_2.ModificationsDB._getModification_1
RuntimeError: the value 'Carbamidomethylation (C)' was used but is not valid; Retrieving the modification failed. It is not available for the residue '' and term specificity 'none'.
const OpenMS::ResidueModification* OpenMS::ModificationsDB::searchModificationsFast(const OpenMS::String&, bool&, const OpenMS::String&, OpenMS::ResidueModification::TermSpecificity) constModification not found: Carbamidomethylation (C)
Warning: OPENMS_DATA_PATH environment variable already exists. pyOpenMS will use it (/usr/local/share/OpenMS/) to locate data in the OpenMS share folder (e.g., the unimod database), instead of the default (/usr/local/lib/python3.10/site-packages/pyopenms/share/OpenMS).