YeoLab / outrigger

Create a *de novo* alternative splicing database, validate splicing events, and quantify percent spliced-in (Psi) from RNA seq data
http://yeolab.github.io/outrigger/
BSD 3-Clause "New" or "Revised" License
61 stars 22 forks source link

Add conda environment for dev #55

Closed willingc closed 7 years ago

willingc commented 7 years ago

This PR adds an environment.yml file that installs a development environment (including conda and pip packages).

The environment channels used will be searched in the following order: conda-forge, r, bioconda. We tend to use conda-forge packages over Continuum's anaconda packages as they are more up to date where Continuum lags due to their release cycle. [Aside: you may wish to upload outrigger to conda-forge at some point.]

This includes dependencies for building docs (sphinx, theme, recommonmark - markdown support, nbsphinx - notebook support).

To use this environment:

conda env create -f environment.yml
source activate outrigger-env
pip install .

To check if installed: outrigger -h

olgabot commented 7 years ago

Great! That's really nice you can specify pip vs conda installers because it really drives me nuts to have the conda_requirements.txt and requirements.txt file be separate.

RE conda-forge - I'm working on adding this to bioconda which is a repository of bioinformatics packages: https://github.com/bioconda/bioconda-recipes/pull/2941 but the build failed because I didn't have pyflakes as a dependency even though it's a dependency of flake8. Sigh. So I'll probably just add pyflakes to the requirements.txt/environment.yml file

willingc commented 7 years ago

Added pyflakes by pip. Let's see if that helps.

willingc commented 7 years ago

Worst case: I think pinning pyflakes would help. pyflakes<1.1,>=0.8.1

olgabot commented 7 years ago

Yeah lint checking is getting too weird... because the pyflakes versions are going to change and it's not relevant to usage or installation. I'm making a separate environment for it in my travis: https://github.com/YeoLab/outrigger/pull/56

On Thu, Nov 17, 2016 at 8:50 AM Carol Willing notifications@github.com wrote:

Worst case: I think pinning pyflakes would help. pyflakes<1.1,>=0.8.1

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/YeoLab/outrigger/pull/55#issuecomment-261301288, or mute the thread https://github.com/notifications/unsubscribe-auth/AAxNcMd79g7dQu7DGSE-XRObp_Qtf0cvks5q_IXKgaJpZM4K1a_N .

willingc commented 7 years ago

@olgabot Good plan 💯

olgabot commented 7 years ago

Awesome, I just added those changes. Can you rebase to master?

olgabot commented 7 years ago

Oh and remove flake8 and pyflakes from requirements... they're causing so much headache