You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Looking good! Most dependencies seem needed and not build up the graph too much (apart from maybe scikit-gstat in xdem, that we might make optional soon).
To summarize what I was saying: The main idea behind making some dependencies optional is to not impose them on your user (like for scikit-gstat imposed by xdem above), and how the dependencies of that other package also create more dependencies (in this case scikit-gstat just requires scipy that's already used, but could be different for other packages), and avoid blowing user's environments bit by bit.
So the choice depends on both on the extra dependencies added by the other package, and on how much that other package is used in your own (i.e. only specific to a single module/functionality, or used across all modules?) and can potentially be left out without affecting the general behaviour of your package.
If the usage of a package is specific to a single functionality (like sliderule is to your asp_plot.altimetry module right now), then it might make sense to make it optional simply by adding something like this at the top of your script:
Would be useful to follow up dependencies with @rhugonnet:
https://github.com/GlacioHack/xdem/blob/main/environment.yml
https://github.com/GlacioHack/xdem/blob/main/dev-environment.yml
https://github.com/SlideRuleEarth/sliderule/blob/main/clients/python/environment.yml
The text was updated successfully, but these errors were encountered: