catalystneuro / roiextractors

Python-based module for extracting from, converting between, and handling optical imaging data from several file formats. Inspired by SpikeInterface.
https://roiextractors.readthedocs.io/en/latest/index.html
BSD 3-Clause "New" or "Revised" License
11 stars 7 forks source link

Remove `HAVE_SCIPY` #316

Open h-mayorquin opened 4 months ago

h-mayorquin commented 4 months ago

We have try-except blocks like this in a couple of places:

try:
    from scipy.sparse import csc_matrix

    HAVE_SCIPY = True
except ImportError:
    HAVE_SCIPY = False

But requirements minimal already has scipy:

https://github.com/catalystneuro/roiextractors/blob/275acbb66672766232fa9f7f1b232d7739d89745/requirements-minimal.txt#L9

So what's the point of them?