Cody Baker

Generalize Validation

14

fix #1431 replaces #1494 by branching from dev instead of the modularity improvement ## Motivation WIP synchronizing the behavior of the CLI `pynwb.validate` function with the library-imported one, namely for...

[Bug]: v2.1.0 ElectrodeTable cols

3

### What happened? Just ran across this in the NWBInspector CI. ### Steps to Reproduce ```python from pynwb.file import ElectrodeTable table = ElectrodeTable( name="electrodes", # default name when building through...

Improve package modularity

4

## Motivation It can be ambiguous to have callable methods in the `__init__` of a package whenever those methods also share a name with a submodule exposed at the same...

related to [#504](https://github.com/NeurodataWithoutBorders/nwb-schema/issues/504) Noticed that the PyNWB documentation for ImageSeries makes no explicit mention of the color channel order (which ought to be RGB). This is somewhat important as we...

### What would you like changed or added to the documentation and why? Just noticed that the public-facing docs https://pynwb.readthedocs.io/en/stable/ and https://pynwb.readthedocs.io/en/latest/ do not match the recent progress (especially new...

[Feature]: if a file passes pynwb.validate, it should be readable

7

### What would you like to see added to PyNWB? Hey all, This came up originally in https://github.com/dandi/helpdesk/discussions/71 that the NWBInspector failed to run on a certain NWBFile. Turns out...

Add conversion factor to waveform columns

1

## Motivation For large amounts of recording data, even adding waveform snippets to the columns of a `Units` table can be a significant task. One step towards reducing unnecessary data...

Fully lazy imports of IO submodules

2

@JuliaSprenger @samuelgarcia Hello everyone - a recent [issue on NeuroConv](https://github.com/catalystneuro/neuroconv/pull/71#issue-1330574689) drew our attention to the fact that global module import structures such as https://github.com/NeuralEnsemble/python-neo/blob/master/neo/__init__.py#L10-L11 and https://github.com/NeuralEnsemble/python-neo/blob/master/neo/io/__init__.py#L278-L379 will actively import every...

[Feature]: Warn to use index argument on call to add_trial (or add_row)

1

### What would you like to see added to PyNWB? From https://github.com/NeurodataWithoutBorders/helpdesk/discussions/71#discussioncomment-8638454 User was confused by HDMF level error during `io.write`. Caused by not declaring a column to be 'indexed'...

[Documentation]: FAQ or Best Practice for representing co-registered ROI IDs

3

### What would you like changed or added to the documentation and why? Opening here for now, but feel free to transfer wherever is best For co-registration of cell IDs...