-
Notifications
You must be signed in to change notification settings - Fork 36
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
add optional inputs to tabulate-seqs
#276
Comments
@gregcaporaso this should probably be closed by #284, but I'll let you look at things and close this or request further changes when you get the chance. |
@Oddant1, @Clockwork-Rat - thanks so much for the work on this. I downloaded and tested #284, and it's working great. A few (little?) requests before we close this one out:
EDIT (one more): |
@gregcaporaso we can make the taxonomies a collection instead of a list. Then they can key them if they want or just leave them unkeyed and the keys will default to numbers. |
Now closed by @Clockwork-Rat |
To assist with assessing why ASVs aren't being assigned taxonomy (one of many forum x-refs), it would be useful if we could optionally provide
FeatureData[Taxonomy]
(and maybe alsoto theFeatureTable[Frequency]
)tabulate-seqs
visualizer. This would add the taxonomy as a new column in the table that is generated bytabulate-seqs
, making it easier to identify the sequences that are getting poor taxonomy assignments.If theFeatureTable[Frequency]
was also provided as input, we could also add the number of times each ASV was observed, and the number of samples each ASV was observed in.Update: rather than taking a
FeatureTable
directly, this action should take feature metadata as input. This allows for arbitrary metadata to be integrated in the resulting visualization, including the number of times each ASV was observed, and the number of samples each ASV was observed in which will be newImmutableMetadata
outputs fromsummarize
following #282. This information is useful for example to help users to prioritize ASVs that are common for further investigation.The text was updated successfully, but these errors were encountered: