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

Bump up number of closest sequences for targeted trees #1711

Merged
merged 5 commits into from
Sep 10, 2024
Merged

Conversation

danrlu
Copy link
Collaborator

@danrlu danrlu commented Sep 10, 2024

The changes in #1709 didn't do much. This is the place we need to update.

Copy link
Contributor

@vincent-czi vincent-czi left a comment

Choose a reason for hiding this comment

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

I assume this makes sense comp bio wise, but I wanted to call out that this will change SC2 in addition to MPX. I expect you're already aware of that, but just wanted to make sure that the change is intended for both pathogens, not just mpox.

@danrlu danrlu merged commit 244c598 into trunk Sep 10, 2024
13 checks passed
@danrlu danrlu deleted the danrlu-patch-1 branch September 10, 2024 18:28
thanhleviet pushed a commit to theiagenghi/theiagenepi that referenced this pull request Sep 12, 2024
…g#1711)

* Update type_plugins.py

* Update type_plugins.py

* Update test_export.py

* Update test_mpx_export.py

* Update test_export.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants