search

MannLabs / alphapeptdeep

Deep learning framework for proteomics
Apache License 2.0
109 stars 21 forks source link

Missing dependencies for nbs #48

Closed straussmaximilian closed 2 years ago

straussmaximilian commented 2 years ago

Hi, I did install alphapeptdeep locally and tried running some of the Notebooks (test_ccs_rt). Here, some dependencies were missing:

wget is in the development requirements.

As a potential solution, one could do one of the following: (1) include lxml in the dev requirements and make a note at the beginning of the notebooks that dev needs to be installed (2) include lxml and wget in the default requirements (3) Download the additional requirements in the notebooks (!pip install lxml)

jalew188 commented 2 years ago

Fixed in v0.1.3: lxml in default req wget in the notebook