churchmanlab / genewalk

GeneWalk identifies relevant gene functions for a biological context using network representation learning
https://churchman.med.harvard.edu/genewalk
BSD 2-Clause "Simplified" License
128 stars 15 forks source link

Bioconda integration #4

Closed chrarnold closed 3 years ago

chrarnold commented 4 years ago

GeneWalker fits very well into Bioconda, do you have plans of adding it to Bioconda as well so that it can be installed via "conda install" also? Would be great, the tools looks very promising!

ri23 commented 4 years ago

Thank you for the suggestion @chrarnold, sounds good, we will look into adding GeneWalk to Bioconda.

bgyori commented 4 years ago

Working on it, https://github.com/bioconda/bioconda-recipes/pull/17469, however, we need to add various dependencies to BioConda first for it to go through.

bgyori commented 4 years ago

It is getting very complicated with dependencies now 4 levels deep needing to be added to BioConda one by one, prospects aren't great of this happening soon.

chrarnold commented 4 years ago

Wow ok I did not know this is so complicated... Was setting up pip much easier? Thanks for the effort, I was under the assumption that adding a recipe to Bioconda is straight-forward?

mfansler commented 4 years ago

@chrarnold I made some YAMLs that can be used to install GeneWalk via Conda, e.g.,

conda env create -f genewalk-latest.yaml

Basically, I install all dependencies that are already available through an Anaconda Cloud channel (Bioconda, Conda Forge, or Anaconda) and then let pip do the rest. Just be aware that best practice is that one should treat such an environment (Conda + Pip mixed) as immutable. If you want to change something in it, edit the YAML and recreate the env.

For the genewalk-latest.yaml, you may need to adjust the ssh to https, depending on how your GitHub access is configured.

bgyori commented 3 years ago

GeneWalk is now integrated with BioConda: https://anaconda.org/bioconda/genewalk (also see https://github.com/bioconda/bioconda-recipes/pull/17469).