spikeinterface icon indicating copy to clipboard operation
spikeinterface copied to clipboard

add hs_detection as sorting component

Open lkct opened this issue 3 years ago • 4 comments
trafficstars

algorithm implementation in hs-detection

lkct avatar Aug 16 '22 22:08 lkct

Just to add some context: This wraps the detection algorithm used in Herding Spikes. It depends on a standalone implementation that's available via pypi. It is fast and parallel, and we'll evaluate/compare performance soon.

mhhennig avatar Aug 17 '22 12:08 mhhennig

Just noticed: The unit tests use Python 3.8, but the new hs-detection package uses 3.9 features. Will it be kept at 3.8 for now, what's the plan?

mhhennig avatar Aug 17 '22 12:08 mhhennig

Just noticed: The unit tests use Python 3.8, but the new hs-detection package uses 3.9 features. Will it be kept at 3.8 for now, what's the plan?

@mhhennig is there a reason why 3.8 is not supported on the hs-detection side?

alejoe91 avatar Aug 23 '22 08:08 alejoe91

Just noticed: The unit tests use Python 3.8, but the new hs-detection package uses 3.9 features. Will it be kept at 3.8 for now, what's the plan?

@mhhennig is there a reason why 3.8 is not supported on the hs-detection side?

@alejoe91 cc @mhhennig We decided at the beginning that the hs-detection package would be based on py3.9 for some new features, and we expect 3.9+ will be majorly used in the following. I've just addressed the issues blocking the tests, so it works (at least with the test setting) on 3.8 now. However, I don't think it's worth enforcing full compatibility to 3.8, involving more code changes and giving up new features.

lkct avatar Sep 06 '22 15:09 lkct

@mhhennig @lkct

Closing this for the moment. Let's discuss over a new PR

alejoe91 avatar Dec 21 '22 10:12 alejoe91