mountainlab-js icon indicating copy to clipboard operation
mountainlab-js copied to clipboard

re-implement ephys.* and ms4alg.* processors using new processor manager

Open magland opened this issue 6 years ago • 4 comments

Use a slightly different namespace, so we can test the equivalence, and then we'll replace the existing with the new.

These include the following:

ephys.bandpass_filter ephys.compare_ground_truth ephys.compute_cluster_metrics ephys.compute_cross_correlograms ephys.compute_templates ephys.convert_array ephys.synthesize_random_firings ephys.synthesize_random_waveforms ephys.synthesize_timeseries ephys.whiten

ms4alg.apply_label_map ms4alg.create_label_map ms4alg.sort

magland avatar Aug 13 '18 11:08 magland

Initial implementation of the aforementioned processors has been completed and is available in 'mlprocessors' branch of respective repositories. Once it is confirmed the ports work as expected, I'll work on improving the implementation.

wysota avatar Aug 17 '18 11:08 wysota

Excellent. Can we merge this into the master branch? Then I will change all my pipelines to use ephys_new.* And if everything seems okay for around a week, we will make the namespace change, and change ephys.* to ephys_old.* and ephys_new.* to ephys.. Then after another week or so we will then delete the ephys_old.

magland avatar Aug 17 '18 12:08 magland

I'd feel safer if you tested it slightly first but if you feel confident about it then go ahead. Once I know it works, I'll fiddle with the implementation as currently everything is in a single file and wraps your original calls. I want to put implementations into dedicated files.

wysota avatar Aug 17 '18 21:08 wysota

Okay, I'll try to get to this soon.

magland avatar Aug 18 '18 01:08 magland