nwb-guide icon indicating copy to clipboard operation
nwb-guide copied to clipboard

Add AbfInterface

Open garrettmflynn opened this issue 2 years ago • 3 comments

This PR adds the AbfInterface to the GUIDE, which passes on the test data.

Note: For the electron file selector, it seems that you aren't allowed to multi-select symlinks (at least symlinked files)—so you have to drag-and-drop the test data into the FileSystemSelector component if you'd like to test more than one

garrettmflynn avatar Oct 16 '23 20:10 garrettmflynn

The source data page here needs work and even from the get-go sends us down the fairly time consuming road of figuring out how to handle the icephys metadata, so let's table that for the time being

CodyCBakerPhD avatar Oct 17 '23 19:10 CodyCBakerPhD

Would it be possible to simply exclude the icephys_metadata field and allow this interface—or does this still leave too much complexity / too little control for the user?

garrettmflynn avatar Oct 17 '23 20:10 garrettmflynn

As indicated in the conversion gallery https://neuroconv.readthedocs.io/en/main/conversion_examples_gallery/recording/abf.html

this metadata is quite important

Icephys in general is much less about the time series data itself and more about all the annotations and structure that goes into describing it

CodyCBakerPhD avatar Oct 17 '23 22:10 CodyCBakerPhD