-
Notifications
You must be signed in to change notification settings - Fork 80
Fix analysis tests #1106
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
Fix analysis tests #1106
Conversation
just pushed from upstream |
Thanks @josenavas, I'm gonna start reviewing right now. @squirrelo, quick reminder that you should also review this one. |
|
||
# write mapping file out | ||
if len(samples) != len(set(samples)): | ||
duplicates = [s for s in samples if samples.count(s) > 1] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can use find_duplicates here: http://scikit-bio.org/docs/latest/generated/skbio.util.find_duplicates.html?highlight=duplicates#skbio.util.find_duplicates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
@josenavas, this looks in really good shape. |
Good to go after @ElDeveloper comments addressed. |
Ready for another round! |
Thanks @josenavas |
Built on top of #1099
The test database was incomplete. We were missing the prep template file and the qiime mapping file for the prep template that was already in the database. I've added those and I fixed the tests for the analysis object.
After this PR, all tests in the qiita_db folder should be passing. However, there will be still some errors on qiita_ware, which I will fix in another PR.