JesseLivezey commented 2 years ago

Description and related issues

There were a few metadata parsing/loading problems that were not being tested on the subset of testing blocks.

Checklist:

[x] All tests pass on catscan: run pytest --basetemp=tmp -sv -n 8 tests on catscan from the root directory
[x] If needed, docs have been update: docs/source has been updated for any added, moved, or removed files
[x] Docs build with no errors: run make clean & make html from the docs folder
[x] No python formatting errors: run flake8 nsds_lab_to_nwb tests from the root directory

Some old tests still fail, but I believe no new tests fail!

JesseLivezey commented 2 years ago

I'll any more changes based on trying to convert the blocks before merging.

jihyunbak commented 2 years ago

Thanks @JesseLivezey ! Looks good to me to here