-
Notifications
You must be signed in to change notification settings - Fork 19
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
time should be conda dependency #23
Comments
Hi, thank you for reporting the issue and sorry for the late reply. You can safely remove time from the script, it is just a remnant from the initial version. I will likely remove time in the next release. |
Any possible workaround to work on a cluster and without building the source? |
I think I can just make a small release this week, it will also incorporate the fix for #21 |
That would be of great help! Thanks! |
Hi @smallfade, SibeliaZ 1.2.3 that does not depend on time should be in bioconda now. |
Hi,
Installed sibeliaz from conda on linux centOS system. Attempted to run with test dataset and receive the following output:
Constructing the graph...
line 131: /usr/bin/time: No such file or directory
line 132: /usr/bin/time: No such file or directory
rm: cannot remove './sibeliaz_test/de_bruijn_graph.dbg': No such file or directory
line 137: /usr/bin/time: No such file or directory
I am running on a shared cluster and therefore do not have permission to install time in /usr/bin. time is installed on the system elsewhere and I updated these lines in the sibeliaz script to reflect the path to installed time.
Then when I run I get
-f: command not found
My guess is that the version of time that is installed is too old and does not have the -f flag.
Then I installed time from conda (https://anaconda.org/conda-forge/time) and changed the path to time in the sibeliaz file to the time installed by conda and the script works.
Would it be possible to add time as conda dependency and update the sibeliaz script to rely on the version of time installed by conda?
The text was updated successfully, but these errors were encountered: