NeuralEnsemble / python-neo

Neo is a package for representing electrophysiology data in Python, together with support for reading a wide range of neurophysiology file formats
http://neo.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
307 stars 240 forks source link

Neuroshare.org no longer exists---drop neuroshare support? #1471

Closed sanjayankur31 closed 3 weeks ago

sanjayankur31 commented 3 weeks ago

neuroshare.org no longer exists, and the python neuroshare package has not seen a release since 2016:

https://pypi.org/project/neuroshare/#history

This indicates that neuroshare is no longer an active project. Should we consider dropping it from neo perhaps?

zm711 commented 3 weeks ago

I would say as long as people have the format and it is acceptable with python/numpy constraints then we can keep supporting it. Unless you see clear problems with the IO, no?

We recently discussed our IO deprecation policy: #1460.

sanjayankur31 commented 3 weeks ago

I wasn't aware that it's a format---the neuroshare pypi page made me think it was primarily to work with the neuroshare.org API, which no longer exists.

Yeh, can be deprecated whenever the policy allows.

zm711 commented 3 weeks ago

https://github.com/NeuralEnsemble/python-neo/blob/60d703337685b734499756d22f45a2f136b61218/neo/io/neuroshareapiio.py#L39

looks like the suffix is .mcd I never use it, but just in case. Thanks for checking though!