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

Replace RAxML tree building with IQ-Tree #46

Closed
j23414 opened this issue Nov 20, 2024 · 1 comment · Fixed by #47
Closed

Replace RAxML tree building with IQ-Tree #46

j23414 opened this issue Nov 20, 2024 · 1 comment · Fixed by #47
Labels
enhancement New feature or request

Comments

@j23414
Copy link
Collaborator

j23414 commented Nov 20, 2024

Context

The West Nile Virus (WNV) pipeline currently uses RAxML for tree building, which is taking up a significant portion of the runtime (about 4 out of 6 hours).

Based on some online reading about RAXML generally runs much slower than IQ-Tree, I checked in with @DOH-LMT2303 and @jameshadfield if we could switch to IQ-Tree or if there was a technical reason why RAXML was selected over IQ-Tree for WNV.

  • @DOH-LMT2303 investigated the literature and found that both RAxML and IQ-TREE have been used in WNV studies. Based on this, she concluded that we could probably use any tool that produces a maximum likelihood phylogeny and get similar results.
  • We noted that Grubaugh lab has switched to using IQ-TREE as the default for their WNV-global build.
  • @jameshadfield confirmed that RAxML was used in the original build because it was the standard at the time. He stated that if he were to create the build today, he would use IQ-TREE.

There doesn't appear to be any WNV-specific reason for using RAxML over IQ-TREE, but let me know if if I'm mistaken. I'm going to explore swapping in IQ-Tree and see if we get any speedups

Description

Swap in IQ-Tree and determine if we get any speedups to further avoid the 6hr GitHub Action runtime limit.

Examples

Possible solution

@j23414 j23414 added the enhancement New feature or request label Nov 20, 2024
@j23414
Copy link
Collaborator Author

j23414 commented Nov 20, 2024

Switching to IQ-Tree (view tree here) reduced runtime drastically:

@j23414 j23414 linked a pull request Nov 20, 2024 that will close this issue
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant