Zach McKenzie

Results 864 comments of Zach McKenzie

Hey @NxNiki, You can use the `exclude_filename` which allows you to search for some files, but not others. This is the opposite of what you want, but can be used...

@samuelgarcia has worked on neuralynx most recently so I would defer to him to comment on adding this feature :) I have no problem with you making the PR, but...

So I don't know openephys very well at all, but I'm wondering if there would ever be a case where the timestamps could be corrupted but the other channels are...

In this issue #1438, someone is asking to ignore corrupted channel and just use samples. If it's too annoying to work around I think that's fine, but it seems like...

Yes it is for the binary/new format. My point is just that some people regardless of the corruption status of their data may want to try to salvage something. That...

Hi @weiglszonja! If that's all it takes then shouldn't that be in the error message? Again, I don't use openephys so I might be saying nonsensical things, but if we...

@NeoKreation, I've been trying to help other people, but we have never been able to get Phy to work on Macs. Phy relies heavily on openGL which has been deprecated...

I guess one question I would have overall is why overload the filename argument? Unfortunately I don't think we would change (like change filename->filenames) since it is a core part,...

I think having `filename` and `filenames` would be super confusing and misused a lot. Would you be okay with `include_filenames` then it would still be `dirname='path'`, `filename=None`, and `include_filenames=['file0', 'file1']`?...

@NxNiki, I agree that's the better overall solution, but changing the argument name could break users' code. So my question for Sam was whether he was okay to do a...