epfl-lts2 / pygsp

Graph Signal Processing in Python
https://pygsp.rtfd.io
BSD 3-Clause "New" or "Revised" License
483 stars 93 forks source link

New sphere graph #55

Open Droxef opened 5 years ago

nperraud commented 5 years ago

@mdeff I am not sure, we should have healpy as a dependency. It is a big package that takes time to install and we only use it for one function. What do you think?

mdeff commented 5 years ago

Absolutely. It should be lazily imported (as matplotlib, pyqtgraph, flann, etc.). That is, it's only imported when the user calls the SphereHealpix constructor, not when the pygsp is imported. The only hard dependencies are numpy and scipy. @Droxef did it correctly.