nwb-conversion-tools

nwb-conversion-tools copied to clipboard

[Feature]: tests take a long time to run locally

2

### What would you like to see added to nwb-conversion-tools? Especially the ones in test_move_utils. Can we make these faster? ### Is your feature request related to a problem? _No...

bendichter

Pre-select channels to load in `CEDRecordingInterface`

2

PR catalystneuro/nwb-conversion-tools#582 introduced the following change: ``` def __init__(self, file_path: FilePathType, smrx_channel_ids: list, verbose: bool = True): changed to -> def __init__(self, file_path: FilePathType, verbose: bool = True): ``` @bendichter...

h-mayorquin

added debugged Dockerfile

1

WIP Builds and runs fine locally, image can currently be pulled from https://hub.docker.com/r/codycbakerphd/test_nwbct Some To-Do's: - [ ] make CatalystNeuro namespace on DockerHub for these kinds of things - [...

CodyCBakerPhD

[Feature]: TIFF interface should take a list of files

4

### What would you like to see added to nwb-conversion-tools? We've seen a couple of times now that people break up a single consecutive ophys recording into multiple TIFF files...

CodyCBakerPhD

Refactor `dict_deep_update` function.

1

While working on PR #481 me and @CodyCBakerPhD were discussing about the poor variable name in this very important function. Right now, we have: https://github.com/catalystneuro/nwb-conversion-tools/blob/5e66722723420856de70e80b8e465f3bb20ebfbd/src/nwb_conversion_tools/utils/dict.py#L121-L124 Which go against the design...

h-mayorquin

Method for extract metada from nwb to make our round trip testing more complete

4

Right now, our ecephys tests with data, the ones in `test_gin_ecephys` mainly compare the traces as written from the recording extractors. https://github.com/catalystneuro/nwb-conversion-tools/blob/a875b40e15ff34d593eb70ac45a188b46b21c216/tests/test_on_data/test_gin_ecephys.py#L197-L220 Whereas the corresponding tests for sorting extractor mainly...

h-mayorquin