search

AllenInstitute / ecephys_spike_sorting

Modules for processing extracellular electrophysiology data from Neuropixels probes
Other
109 stars 91 forks source link

9/create tests #13

Closed jsiegle closed 6 years ago

jsiegle commented 6 years ago

There's a lot in this one -- I will try to keep my PRs smaller in the future. But we now have tests for most of the modules, and the tox config is working on Bamboo. Some of the modules cannot be tested without access to pre-compiled Windows binaries and/or Matlab, so we'll still have to work out a solution for that. But otherwise the organization of the code seems sensible, and only a few tweaks should be required to get this ready for the pipeline.

NileGraddis commented 6 years ago

Looks good to me - thank you for doing this work @jsiegle.