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

New feature: allow merge_taxa to keep recored of merged (and inextension tax_glom). #1707

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

derele
Copy link

@derele derele commented Oct 13, 2023

Hi Paul,

are you interested in adding a (tiny) new feature to phyloseq?

I needed something to track what tax_glom and merge_taxa are doing and implemented a small feature to allow for a function (mostly relevant for pasting together or counting) to be applied to the length>1 entries of the merged/agglomerated taxa.

I exposed the argument I implemented for merge_taxa only to tax_glom, not to any other of the relevant higher level functions (the only other relevant one would probably be tip_glom, I guess).

I left the default intact, so it should not break anything. I added the required documentation (via devtools::document()) and did my best to make sure that devtools::check() complains only about stuff that has nothing to do with my changes.

Feel free to pull or ignore!

All the best,
Emanuel

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.

1 participant