-
Notifications
You must be signed in to change notification settings - Fork 58
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
Crying for help - TPP needs a conda package #108
Comments
@bgruening I've been working on a homebrew recipe. Works on OSX and I'm hoping to get it accepted. I have no experience with conda but maybe the homebrew recipe could be adapted? |
@iracooke thanks I tried this. But conda is way more general and least how we do it. We want to have every dependency defined and we want to run in in environments where perl is not under /usr/bin/perl etc ... maybe we can sit together sometime during the next two weeks and try to fix this? Thanks! |
@bgruening I didn't mean that the galaxy tools should use homebrew but the source code to that recipe shows a modified TPP I made to help with packaging https://github.com/iracooke/tpp . Maybe that is helpful to fix the conda build. I have no idea what the problems are currently with conda. |
@bgruening Yes. I would like to try and help fix this. Maybe something for this weekend. Looking fwd to seeing you in Melbourne 👍 |
Hejhej. |
Yes indeed. Installation of the TPP is a monumental pain. The tools are good in some respects though. As a general rule they are written to be fast and they support a very wide variety of search engines and workflows, not to mention the many statistical ideas they encapsulate. Since they are widely used they are also fairly well tested and validated. There are certainly alternatives out there but many of those have their own problems, or only replace a subset of what the TPP does. So despite it's problems I think the TPP is still relevant. Having said that I'm afraid I haven't got the time to implement a conda recipe. I did get a homebrew recipe accepted into homebrew science though. If someone could adapt that to linux it would be awesome. |
@iracooke Sorry, but I do not understand enough of programming to do the job. I will try to gently push my colleagues, but they are also really buisy. Speaking of the TPP: another drawback is the bad documentation, or do you know good tutorials, especially on the more complex statistics stuff? I did not use half of it, because it was so tedious to find out what everything is really doing and if it's worth the effort. |
If you don't have a specific need for the TPP its probably a good idea to just not worry about it. Getting the tools working well does indeed take a lot of work. I have made a set of ruby wrappers for some of the tools (PROTK ruby gem) but there are many tools I have not wrapped and unless you use MacOS you will still have the installation problem.
…Sent from my iPad
On 21 Apr 2017, at 10:58 pm, Florian Christoph Sigloch ***@***.***> wrote:
@iracooke Sorry, but I do not understand enough of programming to do the job. I will try to gently push my colleagues, but they are also really buisy.
Speaking of the TPP: another drawback is the bad documentation, or do you know good tutorials, especially on the more complex statistics stuff? I did not use half of it, because it was so tedious to find out what everything is really doing and if it's worth the effort.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Most of the tools without testing and a conda package are now TPP based.
I started to work on this here: https://github.com/bioconda/bioconda-recipes/compare/tpp?expand=1
But did not succeed here. Any brave sole that can help me with this gets free beer at next GCC.
We could also think about splitting the different binaries into separate packages and not building the entire thing. Also we might want to build the latest TPP version and upgrade our tools.
This is the last big step missing for a completely tested GalaxyP repository - I think it's worth the pain!
The text was updated successfully, but these errors were encountered: