-
Notifications
You must be signed in to change notification settings - Fork 267
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Review existing dependencies #2007
Comments
Hello @valentynbez Thank you for reviewing the scikit-bio codebase and making these suggestions! Reducing dependencies is a development goal of ours. In the past several months (from 0.5.x to 0.6.0), we have already removed multiple heavy dependencies, including
|
I'd be in favor or a PR to remove |
@wasade Looks like |
Adding a new functionality comes at the cost of complexity and more abstractions. I hereby propose to do a basic review of the current project dependencies.
I.e. a quick search through a codebase reveals:
realsort
fromnatsort
package is usedmunkres
is an outdated and not supported package (last release in 2020)pluggy
andpytest
needed in the deployment version? They can be separated intoscikit-bio
andscikit-bio[dev]
requests
is only used in single file and functionality can be completely moved to urllib3The code that's is dependent on outdated packages should be highlighted, so that if possible replacement for the functionality appears it can be quickly reimplemented. These changes, although cumbersome, will reduce amount of dependencies and ensure software's longevity.
The text was updated successfully, but these errors were encountered: