Zach McKenzie

@huxiaoyi520 sorry a bunch of your message appears to be in Chinese. Is there any way you could display it in English? In general skipping spike waveforms that do not...

@eduardosand, there have been some updates to Neuralynx. Could you try your file after installing neo from source and see if this has now been resolved?

Do we even want that assertion error test? I can change it....-- I think maybe even making something like `NeoWriteError` and `NeoReadError` might be more informative to end users in...

> If you are raising AssertionError for testing things which aren’t assertions about your code’s correctness, e.g. for checking data validity, you are doing it wrong. It is never appropriate...

> One incentive for people to use asserts instead of errors is that it is easier to comply with code coverage metrics because you don't need to really test the...

Hi @neuronzoo, it would be super helpful to see the full script you are trying to run that triggered this error. The `.txt` you shared is indicating that you have...

One more question. I'm looking back at your log you shared and the lines aren't matching up with the current source code. What version of spikeinterface are you using? The...

@neuronzoo Would you willing to upgrade? There have been a lot of fixes (especially for Windows users). I'm wondering if this issue has already been fixed. Installing from source would...

From 97-99 it should be basically all backward compatible. I can't think of any breaking changing. A few functions have been changed to be faster/better, but the structure itself is...

@neuronzoo, Any updates? Just want to see if the update worked. There's actually an even newer version as of today, so if you haven't updated it's an even better time...