search

jupyter-widgets / ipyleaflet

A Jupyter - Leaflet.js bridge
https://ipyleaflet.readthedocs.io
MIT License
1.49k stars 365 forks source link

Reduce number of dependencies #1143

Closed gab23r closed 11 months ago

gab23r commented 11 months ago

I am surpised to see the number of "heavy" dependencies of ipyleaflet. I think that most of them are not required ( scipy, bqplot, geopandas, xarray )

I think it would be an improvement to remove those from the environment.yml, so that user can make a conscious choice to add them or not in their environnent.

martinRenou commented 11 months ago

ipyleaflet only requires ipywidgets, branca, traittypes and xyzservices which are relatively small dependencies and do not pull the packages you mention (see https://github.com/jupyter-widgets/ipyleaflet/blob/master/setup.py#L58).

You are looking at the environment file needed for running the examples on the repository, not the dependencies of the package itself.