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

ENH: support empty files in QIIME1DemuxFormat #159

Merged
merged 1 commit into from
Nov 16, 2017

Conversation

jairideout
Copy link
Member

This change is necessary to support the closed-reference OTU picking API in q2-ninja-ops, which returns a QIIME1DemuxFormat file of sequences that failed to hit the reference. It is possible in some cases for there to be zero failures.

This change is similar to allowing empty FeatureData[Sequence] artifacts, which is required by q2-vsearch's closed-reference OTU picking API.

This change is necessary to support the closed-reference OTU picking API in q2-ninja-ops, which returns a QIIME1DemuxFormat file of sequences that failed to hit the reference. It is possible in some cases for there to be zero failures.

This change is similar to allowing empty `FeatureData[Sequence]` artifacts, which is required by q2-vsearch's closed-reference OTU picking API.
@jairideout
Copy link
Member Author

Ignore coveralls, the coverage drop is artificial.

@thermokarst thermokarst self-assigned this Nov 16, 2017
@thermokarst thermokarst self-requested a review November 16, 2017 14:23
Copy link
Contributor

@thermokarst thermokarst left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks @jairideout!

@thermokarst thermokarst merged commit 0f20f1c into qiime2:master Nov 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants