dieterich-lab / rp-bp

Rp-Bp is a Bayesian approach to predict, at base-pair resolution, ribosome occupancy and translation.
MIT License
7 stars 5 forks source link

Upgrade/Installation #129

Closed eboileau closed 1 year ago

eboileau commented 2 years ago

While changing to CmdStanPy, I upgraded all dependencies/packages (most have "unpinned" versions, however Conda does not necessarily resolve the latest version, we can eventually pin important packages). Now we need to finalise the upgrade and the installation (for both rpbp and pbiotools). I have only worked out the Conda installation (with pip install for pbiotools and rp-bp) as follows:

# clone https://github.com/dieterich-lab/rp-bp.git and https://github.com/dieterich-lab/pbiotools.git
# checkout dev-ssciwr
# install environment under rp-bp
conda env create --prefix /prj/rpbp-dev/bullseye/envs/rpbp-cmdstan -f environment.yml
# then I install pbiotools, followed by rp-bp using each time (editable mode)
pip --verbose install -e . 

Now I have many questions...

samtools: /beegfs/prj/rpbp-dev/bullseye/envs/rpbp-cmdstan/bin/../lib/libtinfow.so.6: no version information available (required by samtools)
samtools: /beegfs/prj/rpbp-dev/bullseye/envs/rpbp-cmdstan/bin/../lib/libncursesw.so.6: no version information available (required by samtools)
samtools: /beegfs/prj/rpbp-dev/bullseye/envs/rpbp-cmdstan/bin/../lib/libncursesw.so.6: no version information available (required by samtools)

Listed in #133

lkeegan commented 2 years ago

Under rp-bp I ran pytest tests/ successfully, but latest commit doesn't work, same problem with pbiotools. Something is wrong with the python version(s), and with pip installation...

I am confused now why we mix Conda and pip... ideally, we want ONE Conda install (environment with dependencies + rpbp/pbiotools, and pip only for packages/version not on Conda). We still put rp-bp and pbiotools on PyPI, but in that case users will have to handle dependencies by themselves (Flexbar, STAR, CmdStan, etc. ). When this is sorted, we can also create containers (Docker/Singularity) for the full Conda install.

I agree - and everything is now on bioconda so there can be a pure conda install. There are also already docker/singularity containers for pbiotools from bioconda (see https://github.com/dieterich-lab/pbiotools/pull/9), once rp-bp is packaged in the same way on bioconda there would also be containers available for that.

eboileau commented 1 year ago

I just removed the pins, except for pysam. The latest version on PyPI is 0.20.0 (28 Oct), but hasn't been updated on bioconda yet.

I need to clean the requirement/environment list, in particular if we do some more cleaning (remove unused scripts, etc.)

eboileau commented 1 year ago

Pysam 0.20.0 is now on Conda. The environment file has been updated.