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

Bioconda package #5

Closed
muslih14 opened this issue Oct 26, 2017 · 7 comments
Closed

Bioconda package #5

muslih14 opened this issue Oct 26, 2017 · 7 comments

Comments

@muslih14
Copy link

I'd like to congratulate the developers on a great metagenomics software tool. I'd like to recommend the developers add simka to the bioconda channel as a package as it would enable easier installation and adoption by the community.

Best wishes,
Muslih.

@nick-youngblut
Copy link

There is a simka bioconda recipe, which seems to have existed for a little while now, which makes me wonder whether this repo is still maintained...

There is an issue with the bioconda recipe, which stems from how simka is implemented. More specifically, if the user tries to run simka from the conda env (simka is in the PATH), then simka complains:

EXCEPTION: Unable to get the real path for 'simka'

Why is the real path necessary?

@lecorguille
Copy link

Indeed, I get the same message if I use simka from the $PATH without absolute or related path.
It seem that simka need to know where it's to find some other alongside scripts, binaries or libraries?

If you use this:

$(which simka) -in example/simka_input.txt -out /tmp/results -out-tmp /tmp/temp_output

It works!

@nick-youngblut
Copy link

@lecorguille any progress on fixing this simka path issue? $(which simka) is a confusing hack that each user has to figure out.

@lecorguille
Copy link

lecorguille commented Nov 18, 2020

No idea. It's a "bug" that have to be figure out by the sinka dev team.
I'm afraid that I'm enough confortable with c++ to tackle the issue myself.

They should implement either a way to add all their side binaries in the $PATH or prepend their side binaries calls with a sort of realpath $(dirname $0) (I ❤️ bash)

@clemaitre
Copy link
Collaborator

Thank you @nick-youngblut and @lecorguille for bringing this issue back to the forefront. I had forgotten about it and this is indeed an important issue to make simka easier to use. Now that we have at last some human resources on simka, we will try to fix this soon.

Best,
Claire

@nick-youngblut
Copy link

Great! As an FYI on simka: I just used simka to identify a few outlier samples in my metagenome dataset. simka is especially helpful when most reads don't map to reference databases, so such ref-based methods may not reveal outliers.

@clemaitre
Copy link
Collaborator

This issue has been fixed in release 1.5.3 (commit 3bac22d)

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

No branches or pull requests

4 participants