-
Notifications
You must be signed in to change notification settings - Fork 1
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
IQtree is locked to v1.6 rather than something modern like v2.2 #7
Comments
Pinning to Users may need to install these in their separate environment. We use Nextclade in place of pangolin/pangolearn etc, so it makes sense to remove from standard environment. |
Hmm. At the beginning of this year we explicitly added pangolin and pangolearn to our Docker runtime (and our ambient runtime install instructions). This is why they're also in our newer Conda runtime here (defined by the
Is that a change from earlier this year?
Their deps are pretty unfriendly to coexistence, but I'm not sure it means we shoudn't use them. |
IIRC, they were added before Nextclade could annotate pango lineages. Back then, we thought it may be worth running Usher ourselves so we have up to date lineage annotations (there were some GISAID delays, and GISAID used pangoLEARN). In the end, we never ran pangolin (Usher or pangoLEARN) in ncov-ingest. Happy to hear feedback, whether anyone actually uses pangolin/pangoLEARN from our docker images/conda environment. Not including Usher makes it not very practical though. |
Our ncov workflow still nominally supports running Pangolin when the
|
I found a few uses using GitHub Code Search:
As well as at least one indication of "dark" usage (e.g. usage we can't directly find): |
Current Behavior
Currently both osx and linux lockfiles install IQtree v1.6.
One way we could end up with lower versions is if a package moved from conda-forge to bioconda. Then we're stuck with whatever the last version was that was in conda-forge.
The text was updated successfully, but these errors were encountered: