Zach McKenzie

This looks fine to me. Are there any test files that could help us double check that adding just there works (other than seeing that it seemed to work in...

Merged :) and just reran tests.

@h-mayorquin, Just reading this now, but my understanding is that yes the rescaling is a patch on top in that `get_spike_timestamps` returns whatever the file format returns (sometimes seconds, microseconds,...

Want to ping @h-mayorquin for this. Do you want more discussion around this? Or are we okay to close. Again simple answer is: get_*_timestamps: returns the raw data in whatever...

@cboulay, test files were merged, but now you have some merge conflicts 1) we've run black on the code base and 2) we switched over to f-strings rather than %...

@cboulay, We are planning on including this in the next release which will go out in the next few days so if you have the time to either implement my...

Normally there is a little check box on the right side of the page just below all the info. When I first looked at your PR you had it turned...

Seems like it failed. Could you doublecheck the gin link I gave you and make sure it has all the files necessary for blackrock?

So I think there are two paths that need to be fixed. One at the top and one at the bottom. Since I can't push stuff here do you want...

That's cool. One last check: could you try this as the filename? Previously you didn't have the -001 right? so maybe it needs the full file name for the .nev...