-
Notifications
You must be signed in to change notification settings - Fork 20
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 tabulate-counts
action
#151
Conversation
Seems good pending CI, this is super helpful to have! |
Thanks @wasade! These failures seem unrelated, so we'll get those sorted out independently of this PR. |
I'm thinking a better name for this would be |
Linting should pass after #152 is merged. |
@gregcaporaso I fixed the merge conflicts is this ready to merge? |
Yep, this is ready to go! |
I just added a note to the draft release notes on this one. |
This adds a new action,
tabulate-counts
, which reads one or more demux artifacts and generates anImmutableMetadata
artifact with the per-sample sequence counts.Fixes #150.