Added nf-test for kat/hist #6532
Conversation
|
I'm wondering if this module should be deprecated now. I don't think Kat is maintained anymore and there's both Merqury and MerquryFK now. |
Thanks @mahesh-panchal The conda is also failing. If we agree to deprecate it, I'll remove the conda, add the deprecation message and ping you again for another look. |
|
Do you know why conda is failing? I can't see what the error message is from the PR logs, but running in Gitpod, it's to do with needing to use the classic conda solver. Bernardo hasn't made a release since 2018, and while the issues do seem active on the repository, I think the tool is in maintenance mode at best. The problem is, I'm also not sure how to do the deprecation note. Merqury is the maintained version of the tool, but only does a subset of what KAT does. MerquryFK does more, but isn't really maintained. |
It says,
It might be a memory related issue. But then docker and singularity have the same amount of memory. I'll try to pin the dependency versions to see if that helps. |
|
Yeah, I don't know how to solve that. I've never come across it before. |
GallVp
force-pushed
the
nf-test/kat_hist
branch
from
8457b2b to
080a251
Compare
| - conda-forge::python=3.8.5 # Locked with the container | ||
| - conda-forge::numpy=1.19.1 # Locked with the container | ||
| - conda-forge::scipy=1.5.2 # Locked with the container | ||
| - conda-forge::matplotlib=3.3.1 # Locked with the container |
There was a problem hiding this comment.
Do you need to specify these separately here? Are they not included in the bioconda::kat environment anyway?
What does # Locked with the container imply?
Added nf-test for kat/hist with #6226