Skip to content

feat: import/export of mzTab-M file from Summarized Experiment - #72

Merged
gabrieletome merged 5 commits into
develfrom
gabri
Jul 16, 2026
Merged

feat: import/export of mzTab-M file from Summarized Experiment#72
gabrieletome merged 5 commits into
develfrom
gabri

Conversation

@gabrieletome

Copy link
Copy Markdown
Collaborator

No description provided.

@gabrieletome
gabrieletome requested a review from philouail July 13, 2026 07:52

@philouail philouail left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's great, I had just a few commetns :) thank you very much !!

Comment thread vignettes/generation_of_mzTabM_file.qmd Outdated
Comment thread vignettes/generation_of_mzTabM_file.qmd Outdated
end-to-end metabolomics data workflow in mzTab-M format, based on the
Metabolights data set *MTBLS8735*. The original *xcms*
preprocessing result object is available in Metabonaut and is included also
within the *RmzTabM* package. After collecting all necessary experimental

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what do you mean "is included also
within the RmzTabM package." ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We use this same example in RmzTabM, so we include it inside the package.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would remove :)

Comment thread vignettes/generation_of_mzTabM_file.qmd
Comment thread vignettes/generation_of_mzTabM_file.qmd Outdated
The reconstructed `SummarizedExperiment` contains the same measurement data.
Differences in `rowData` and `colData` reflect that only fields present in the
SMF and MTD sections are loaded and that column names follow the mzTab-M
specification.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would maybe add a bit of a "outlook" part discussing what using a standardized results format brings to the field and maybe also mentioning that in the future we will also be able to export directly from the xcms object in a similar way ? As you want. it doesn't have to be long but we need to motivate why people would want to do this.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's perfect !!

Comment thread DESCRIPTION
@gabrieletome
gabrieletome requested a review from philouail July 14, 2026 14:10

@philouail philouail left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing thank you !

The reconstructed `SummarizedExperiment` contains the same measurement data.
Differences in `rowData` and `colData` reflect that only fields present in the
SMF and MTD sections are loaded and that column names follow the mzTab-M
specification.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's perfect !!

@gabrieletome
gabrieletome merged commit 5f986f7 into devel Jul 16, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants