-
Notifications
You must be signed in to change notification settings - Fork 249
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Request : Add compatibility with next gen BRW files #1156
Comments
Hi @BptGrm, |
@alejoe91 Do you have an idea on how to extend this? |
Unfortunately not. I think that one will need to explore the new h5 test files and see where they save the relevant stuff. It would be great to ask support from someone from 3Brain directly. |
@BptGrm Any news on this from your side? As mentioned above, feel free to contribute a small test dataset to GIN to start extending the neo IO. |
Hi, thanks for bringing this up again ! I submitted a very basic PR to the HerdingSpikes spike sorter to read newer BrainWave files. This might be a starting point but it's definetly not compatible with the multi-well recording that this format is meant to support. I dug pretty deep in the file structure at the time but there were still some aspects that I couldn't quite figure out, namely how they manage data chunks. I tried reaching out to 3Brain about this issue a while ago, but I didn't get any answers. As I mentionned in the original post of this issue, they did send me some documentation on the format that I could send through email. I have a few recordings in this format that I could share, I'll get started on the GIN upload procedure soon. |
Hi @BptGrm Thanks for the update. This sounds like it will still take some time and a lot of tinkering to figure out the new format without someone from 3Brain helping out. Unless you require neo supporting the new format I would say we postpone this until 3Brain contributes or someone knowing the format in more detail has a chance to look at it. |
I'll ask 3Brain again and see if I can get a more positive answer. In the meantime, if anyone desperately need to open newer BrainWave file through neo, there's another open source project called Xenon LFP analysis which contains a downsampling tool that can convert those files to the older format. Setting the 'new' sampling rate to the original should do the trick. |
Hi all, |
@Lychfindel As of #1326 it is now supported. |
@Lychfindel you need to install NEO from sources! |
Thank you! This is a great news! |
Looking through the docs again, those formats are quite a bit more complicated. |
In principle it should be possible to load the other compressed formats in NEO.
|
I was not aware that was possible to load spike times and waveforms, this will definitely work for the I agree with you that is always better storing raw data, unfortunately I'm working with a dataset recorded in the past by a different research group. In any case, I completely understand your point to support only raw formats! |
Coming back to this, I have a working version of the I can open a "work in progress" PR if anyone is interested.
|
I only now saw this ongoing discussion and already implemented the reading of It tightly follows the code example given in the 3Brain documentation. We could also make some of the options available as parameters to the init function of the |
Hi,
3Brain is currently moving towards BRW v4.x, compatible with newer versions of their software but most importantly adapted to multiwell chips, which means that recordings from multiple parallel chips can be stored in the same file.
This implies a major rework of the underlying HDF5 architecture, and the current
BiocamIO
reader is not compatible.I got the OK from 3Brain to share their documentation on that file format, but it would still probably be better to not have it publically hosted.
Is there someone I can reach out to that could take a look at that documentation and possibly figure out how to update the reader ?
Thanks !
The text was updated successfully, but these errors were encountered: