Skip to content
This repository has been archived by the owner on Nov 9, 2023. It is now read-only.

removes dependence on cogent's Sequence object #1503

Merged
merged 3 commits into from
Apr 11, 2014

Conversation

gregcaporaso
Copy link
Contributor

there are still dependencies in the denoiser code, which is going to go away, and in filter_alignment.py, which is being rewritten (#1500).

NOT READY FOR MERGE

@ghost
Copy link

ghost commented Apr 11, 2014

Build results will soon be (or already are) available at: http://ci.qiime.org/job/qiime-github-pr/815/

@gregcaporaso
Copy link
Contributor Author

@ElBrogrammer, this is now ready for review. It's a small one, and the tests passed prior to the merge of #1497 when this contained all of the commits from #1497, so I think it's safe to merge without waiting on the tests.

@jairideout
Copy link
Member

Yeah, we all know how well that strategy works out for me... ;)

jairideout added a commit that referenced this pull request Apr 11, 2014
removes dependence on cogent's Sequence object
@jairideout jairideout merged commit 92181b1 into biocore:master Apr 11, 2014
@jairideout jairideout self-assigned this Apr 11, 2014
@gregcaporaso gregcaporaso deleted the remove-cogent-Sequence branch December 19, 2022 16:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants