compomics / DeepLC

DeepLC: Retention time prediction for (modified) peptides using Deep Learning.
https://iomics.ugent.be/deeplc
Apache License 2.0
52 stars 18 forks source link

add scikit-learn dependency #38

Closed paretje closed 2 years ago

paretje commented 2 years ago

675f5ab676e33ed4f7eee9824a830417f647664e introduces the use of sklearn, but doesn't add it to the dependencies. As the new deeplc.trainl3 is always imported, 0.2.0 currently doesn't work unless you also install scikit-learn. Alternatively, we could make the import optional, in which case we could make scikit-learn an optional dependency.