-
Notifications
You must be signed in to change notification settings - Fork 35
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
additional mock communities to include #26
Comments
These datasets do not have barcode files. Should we change the travisCI requirements to eliminate the barcode file requirement? |
Yes, I think so. Can we adapt the requirement in some way so there is either a barcodes file or some other scenario (I'm not able to download the files at the moment due to slow wifi connection, so I don't yet have a suggestion for what that requirement could be). |
Thanks! perhaps we can require a valid barcodes fp if barcodes are listed in the mapping file, or if > 1 sample is listed in the mapping file. |
That should work. Want to make that change? I can review tomorrow when I add the others to Dropbox. |
Both of these communities (and others) were added with PR #52 . Resolving. |
From Robert Edgar (via feedback on the PeerJ pre-print):
The text was updated successfully, but these errors were encountered: