Skip to content
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

Support for TreeSummarizedExperiment #802

Open
TuomasBorman opened this issue Nov 20, 2024 · 5 comments
Open

Support for TreeSummarizedExperiment #802

TuomasBorman opened this issue Nov 20, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@TuomasBorman
Copy link

Description of feature

Hello!

Currently, the output options include phyloseq. In Bioconductor, there is emerging data container, TreeSummarizedExperiment (TreeSE), that harness the common SummarizedExperiment object. Support for TreeSE would benefit both TreeSE ecosystem and ampliseq community.

More information on TreeSE ecosystem you can find from here: https://microbiome.github.io/OMA/docs/devel/

-Tuomas

@TuomasBorman TuomasBorman added the enhancement New feature or request label Nov 20, 2024
@d4straub
Copy link
Collaborator

Dear Tuomas,

that does indeed sound interesting, would you be able to open a PR with a module that produces such an object?

@TuomasBorman
Copy link
Author

I can have a look, however, it might take some time due to other tasks. I will come back to you by the end of the year.

@hindrek
Copy link

hindrek commented Nov 27, 2024

Hi @TuomasBorman

Thanks for the good initiative!

Have you considered creating a standalone module for TreeSummarizedExperiment, which could benefit not only nf-core/ampliseq but possibly other nf-core pipelines as well?

For example, SummarizedExperiment has a nf-core module, which can be easily integrated to any pipeline: https://nf-co.re/modules/summarizedexperiment_summarizedexperiment/

Hindrek

@TuomasBorman
Copy link
Author

Hello @hindrek!

My knowledge on nf-core is unfortunately too little, but that sounds something that should be considered.

TreeSummarizedExperiment is versatile data container that can be used anywhere instead of phyloseq (metagenomic or amplicon data). TreeSE can be used also for other datatypes that have hierarchical structure that can be represented with tree. For data that do not include hierarchical structure, SummarizedExperiment would make sense (TreeSE inherits SE). It is the most widely used data container in Bioconductor, optimized for biological data in general that has abundance table.

@d4straub Should TreeSE be supported more widely in nf-core?

@d4straub
Copy link
Collaborator

Yes, a nf-core module would be optimal to reach potentially more users. However, it is pretty beneficial to first have working code before doing so, and that what we also create here by testing, imho. Putting TreeSE to nf-core also adds an overhead compared to a local module (when considering only nf--core/ampliseq development).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants